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.

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

Should #CounterTest be packaged in MyCounter-Test instead of MyCounter? @ 0638b16 (PDF) #48

Open
offray opened this issue Feb 5, 2020 · 1 comment

Comments

@offray
Copy link

offray commented Feb 5, 2020

Following the subsection "5.4 Define a Test Case" the learner gets a warning by the code assistant, saying that the "Test class not in a package with name ending with '-Test'". Because this is a book for Pharo novices, such warning could raise some concern on learners.

Should the example code reflect the suggestions of the code assistant to reduce warnings?

@Ducasse
Copy link
Member

Ducasse commented Feb 5, 2020

Yes definitively Can you open a ticket in Pharo bugtracker so that we can produce a list of rules for a novice profile.

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

No branches or pull requests

2 participants