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.

Sign up for GitHub

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

Jump to bottom

Ordered Hashtables .Keys and .Values properties aren't regular arrays #124

Closed
thomasrayner opened this issue Jan 7, 2019 · 0 comments
Closed

Ordered Hashtables .Keys and .Values properties aren't regular arrays #124

thomasrayner opened this issue Jan 7, 2019 · 0 comments
Labels
Category-Koans Invoking the Great Doubt Good First Issue ❇️ Should be quick and easy! Issue-Bug 🐛 Something's wrong! Up For Grabs / Hacktoberfest 💻 If it's Up For Grabs, take it and run with it! If not, ask if it's in progress / you can take it.

Comments

@thomasrayner
Copy link
Contributor

Describe "Koan Bug, Issue, or Help Request"

https://github.com/vexx32/PSKoans/blob/master/PSKoans/Koans/Foundations/AboutHashtables.Koans.ps1#L137

Context "Your Attempts"

The .Keys and .Values properties of ordered hashtables aren't regular arrays and so these assertions fail, even when the learner enters the values in order.

Context "Additional Information"

Appending .ForEach{$_} to both lines to get an array out of these properties would work around this issue. Otherwise the arrays will have to be constructed differently.
@vexx32 vexx32 mentioned this issue Jan 7, 2019
Open
@vexx32 vexx32 assigned vexx32 and unassigned vexx32 Jan 9, 2019
@vexx32 vexx32 added Issue-Bug 🐛 Something's wrong! Up For Grabs / Hacktoberfest 💻 If it's Up For Grabs, take it and run with it! If not, ask if it's in progress / you can take it. Good First Issue ❇️ Should be quick and easy! Category-Koans Invoking the Great Doubt labels Jan 9, 2019
@vexx32 vexx32 added this to the v0.43.0 milestone Jan 9, 2019
vexx32 added a commit that referenced this issue Mar 29, 2019
@vexx32
🔧 Fix outstanding koan issues
4169ba0 
🔧 Fix #129
🔧 Fix #125
🔧 Fix #124
@vexx32 vexx32 closed this as completed in 2dd2f60 Mar 30, 2019
@tomlarse tomlarse mentioned this issue May 10, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Category-Koans Invoking the Great Doubt Good First Issue ❇️ Should be quick and easy! Issue-Bug 🐛 Something's wrong! Up For Grabs / Hacktoberfest 💻 If it's Up For Grabs, take it and run with it! If not, ask if it's in progress / you can take it.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thomasrayner @vexx32