Skip to content

Improve find value specs#7

Open
jelaniwoods wants to merge 1 commit into
mainfrom
jw-update-find-value-specs
Open

Improve find value specs#7
jelaniwoods wants to merge 1 commit into
mainfrom
jw-update-find-value-specs

Conversation

@jelaniwoods
Copy link
Copy Markdown
Contributor

Problem

Yet my terminal provides different solutions compared to what is being asked for:

E.g. answer from terminal:

"Could not find the integer 22"

Expected result:

"22 is under the key e"

To me it seems that the test logic is correct, but the on-screen wording confused me a lot..

https://piazza.com/class/l8k75wwlzfp3bq/post/43_f1

The current tests rely on the random numbers too much and makes it hard for students to verify that they're program is doing the right thing.

Solution

Update tests to use hard coded values that are present in the hash instead of a random one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant