water / mainline

Core web service
GNU Affero General Public License v3.0
9 stars 0 forks source link

Testa git hooks #158

Closed Tarrasch closed 12 years ago

Tarrasch commented 12 years ago

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".

Tarrasch commented 12 years ago

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 commented 12 years ago

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 commented 12 years ago

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 commented 12 years ago

@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 commented 12 years ago

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 commented 12 years ago

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

Tarrasch commented 12 years ago

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