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

Testa git hooks #158

Closed
Tarrasch opened this issue May 4, 2012 · 7 comments
Closed

Testa git hooks #158

Tarrasch opened this issue May 4, 2012 · 7 comments
Assignees

Comments

@Tarrasch
Copy link
Contributor

Tarrasch commented May 4, 2012

Känns lite som en spin-off på #141.

Detta är alltså fortfarande tester på git push, fast det som berör hooksen, alltså att water svarar "grattis, vi skicka in din submission".

@ghost ghost assigned Tarrasch May 4, 2012
Tarrasch added a commit that referenced this issue May 4, 2012
Tarrasch added a commit that referenced this issue May 25, 2012
Tarrasch added a commit that referenced this issue May 25, 2012
@Tarrasch
Copy link
Contributor Author

Nu finns två tester som båda passerar, kvar står nu att skapa lite fler tester (bara allmänt bra), men främst tester som testar resubmissions. Skulle vi köra #resubmit för det?

@Tarrasch
Copy link
Contributor Author

Ok, tänker ni git-CLIt ska fungera såsom specarna anger just nu? Finns det något mer beteende som kan tänkas behövas täckas? Vad säger ni @water/owners ?

@jesjos
Copy link
Member

jesjos commented May 27, 2012

Det ser bra ut. Hur långt har du kommit med implementeringen?
Detta failar just nu vid "handtestning":

it "puts's lab has group in pending" do
      lab_has_group.should be_initialized
      push_new_commit "#submit"
      lab_has_group.should be_pending
end

@Tarrasch
Copy link
Contributor Author

@jesjos du missförstod mig. Bara hälften av testerna passerar nu, jag ville mera att ni kollar och tycker specarna ser ok ut.

För presentationen: Är det viktigt att få det där testet att passera? Jag kommer vara afk ett bra tag tills ikväll, om det är viktigt inför presentationen så ring mig.

@jesjos
Copy link
Member

jesjos commented May 27, 2012

Nej jag missförstod inte alls dig. Jag sade att specarna såg bra ut. Med tanke på att jag förstod att du skriver specarna först och implementerar sen så frågade jag hur det går med implementeringen, eftersom den nämnda specen hade behövts för presentationen.
Det hade gärna fått fungera, men jag kan fixa state-bytet i konsolen och klippa i screencasten, så du behöver inte släppa allt och fixa det.

@Tarrasch
Copy link
Contributor Author

Aha ok, du förstod ändå, aja, ja, fixa state-byten i konsolen så ordnar jag den riktiga logiken senare.

@Tarrasch
Copy link
Contributor Author

Så, nu har jag specat alla viktiga fall jag kommer på och de passerar. Hurra! :)

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