Skip to content

Change expected console log output in test#854

Open
jaimereymendoza wants to merge 1 commit into
hexlet-basics:mainfrom
jaimereymendoza:patch-1
Open

Change expected console log output in test#854
jaimereymendoza wants to merge 1 commit into
hexlet-basics:mainfrom
jaimereymendoza:patch-1

Conversation

@jaimereymendoza

Copy link
Copy Markdown

the exercise says you need to return 's' but the test expects 'grip'

the exercise says you need to return 's' but the test expects 'grip'
@fey

fey commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Thanks for catching this inconsistency! 🙏

You spotted a real problem, but the mismatch is actually in the localized task texts, not in the test. The reference solution index.js builds its output from text = 'Python programming' at indexes 10, 8, 15, 7, which produces grip — so expect(firstArg).toBe('grip') is correct. Changing the test to 's' would make the reference solution fail.

The real cause: the en/es EXERCISE.md (and the legacy description.es.yml) still described an older version of this lesson (Na\nharis → print s), out of sync with the ru text, index.js, and test.js, which are all the "grip" exercise.

This has been fixed correctly in #855 by updating the stale en/es task texts (test and code left unchanged).

Leaving this PR open on purpose as a reminder to verify once the updated lesson is published on the platform.

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.

2 participants