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

Alma: Reqs done #64

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Alma: Reqs done #64

wants to merge 17 commits into from

Conversation

mimyal
Copy link

@mimyal mimyal commented Sep 9, 2016

My challenges this week was a) putting tests into the correct class b) over-reliability on copy/paste c) trouble shooting (I feel I have a start in how to go ahead from now) d) trusting old code when new things break. And some more things. Probably, my brain shutting down.

I learnt a few tricks in how minitests work and have a beginner tool kit in what to do when things break and I don't know why. I was very frustrated at not being able to easily test the self.files until I had it explained to me that it is one of the hardest things to test.

Copy/Paste made it hard to TDD because it tricks me to believe I tested the code already.

Copy/Paste was good for forcing in new information. In the same way as they have Kindergarteners chant 10, 20, 30, 40, 50, 60, 70, 80, 90, 100 (!) until even the non-English speakers speak it.

I refactored using enumerables which made my last hour or so quite satisfactory.

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