Skip to content
This repository has been archived by the owner on Jun 1, 2024. It is now read-only.

Error if dataprovider keys include periods #80

Open
fredemmott opened this issue May 7, 2019 · 1 comment · May be fixed by #117
Open

Error if dataprovider keys include periods #80

fredemmott opened this issue May 7, 2019 · 1 comment · May be fixed by #117

Comments

@fredemmott
Copy link
Contributor 

Fatal error: Uncaught exception 'OutOfBoundsException' with message 'Out of bounds vec access: invalid index 2' in /Users/fredemmott/code/user-documentation/vendor/hhvm/hacktest/src/HackTestCLI.hack:62
@fredemmott
Copy link
Contributor Author

Might not be needed, now that dataproviders are no longer encoded. Will leave open until there's a unit test.

lexidor added a commit to lexidor/hacktest that referenced this issue May 25, 2023
@lexidor
Unit test with period in provider key
808a0c8 
Confirms that hhvm#80 is fixed.
@lexidor lexidor linked a pull request May 25, 2023 that will close this issue
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Unit test for dataproviders with period in key lexidor/hacktest
1 participant
@fredemmott