The Wayback Machine - https://web.archive.org/web/20201211145348/https://github.com/tc39/test262/pull/2366
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unnecessary "includes" directives #2366

Merged
merged 2 commits into from Sep 27, 2019
Merged

Conversation

@jugglinmike
Copy link
Contributor

@jugglinmike jugglinmike commented Sep 25, 2019

The values defined by the referenced files are not used by these tests.
This makes their inclusion superfluous, which needlessly increases the
time to execute the tests and may confuse some readers.

Also,

Update tests to use APIs defined by included files

The $DETACHBUFFER function is the preferred mechanism for detaching array buffers.

Likewise, verifyNotWritable is the preferred mechanism for asserting lack of writability.

jugglinmike added 2 commits Sep 24, 2019
The values defined by the referenced files are not used by these tests.
This makes their inclusion superfluous, which needlessly increases the
time to execute the tests and may confuse some readers.
The `$DETACHBUFFER` function is the preferred mechanism for detaching
array buffers.

Likewise, `verifyNotWritable` is the preferred mechanism for asserting
lack of writability.
@leobalter leobalter merged commit e2903e2 into tc39:master Sep 27, 2019
2 of 8 checks passed
2 of 8 checks passed
ci/circleci: ChakraCore: New or modified tests execution Your tests failed on CircleCI
Details
ci/circleci: JSC: New or modified tests execution Your tests failed on CircleCI
Details
ci/circleci: SpiderMonkey: New or modified tests execution Your tests failed on CircleCI
Details
ci/circleci: V8 --harmony: New or modified tests execution Your tests failed on CircleCI
Details
ci/circleci: V8: New or modified tests execution Your tests failed on CircleCI
Details
ci/circleci: XS: New or modified tests execution Your tests failed on CircleCI
Details
ci/circleci: Project lint and generation unit tests on Python 3 Your tests passed on CircleCI!
Details
ci/circleci: Project lint, generation tests and build Your tests passed on CircleCI!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.