-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Labels
refactorAn improvement to existing test/module codeAn improvement to existing test/module code
Description
Description
Currently the random test suites are dependent on deprecated functionality:
var arr;
arr[i] = val;This feature is already explicitly being testing in the basic array test suite. If this feature fails having this in an unrelated test suite could cause for undo time spent seeking a problem in unrelated tests. This change only serves the purpose to make debugging slightly easier in the future.
I had already summitted a PR relating to this, but only realized now that I should have made a ticket for this first for discussion. Apologies.
#268
Script Name
BasicRandomTestSuite
Steps To The Task
- Remove arbitrary array declarations from a test suite not intended to test those.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
refactorAn improvement to existing test/module codeAn improvement to existing test/module code