Moved ENABLE_CONTRACT_CHECKING to the top of new_setattr

[Bruce-8]

Motivation and Context

This pull request is made so that any contract or type checks are not executed if ENABLE_CONTRACT_CHECKING is False.

Your Changes

Description: Moved the check for ENABLE_CONTRACT_CHECKING to the top of the function so that the new_setattr function doesn't execute any contract or type checks if the constant is False.

Type of change (select all that apply):

• Breaking change (fix or feature that would cause existing functionality to change)

Testing

• Added test case to ensure that no error is raised if a type check is violated (in the case that ENABLE_CONTRACT_CHECKING is set to False).

Questions and Comments (if applicable)

Checklist

• I have performed a self-review of my own code.
• I have verified that the CI tests have passed.
• I have reviewed the test coverage changes reported on Coveralls.
• I have added tests for my changes.
• I have updated the CHANGELOG.md file.

[coveralls]

Pull Request Test Coverage Report for Build 5686178850

• 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.005%) to 94.448%

---

Totals
Change from base Build 5683506524:	0.005%
Covered Lines:	3266
Relevant Lines:	3458

---

💛 - Coveralls