Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(195)

Delta Between Two Patch Sets: state/state_test.go

Issue 6111053: Adopted needs-upgrade change of Python code (Closed)
Left Patch Set: Created 12 years, 11 months ago
Right Patch Set: Adopted needs-upgrade change of Python code Created 12 years, 11 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « [revision details] ('k') | state/unit.go » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 // launchpad.net/juju/go/state 1 // launchpad.net/juju/go/state
2 // 2 //
3 // Copyright (c) 2011-2012 Canonical Ltd. 3 // Copyright (c) 2011-2012 Canonical Ltd.
4 package state_test 4 package state_test
5 5
6 import ( 6 import (
7 "fmt" 7 "fmt"
8 . "launchpad.net/gocheck" 8 . "launchpad.net/gocheck"
9 "launchpad.net/gozk/zookeeper" 9 "launchpad.net/gozk/zookeeper"
10 "launchpad.net/juju/go/charm" 10 "launchpad.net/juju/go/charm"
(...skipping 815 matching lines...) Expand 10 before | Expand all | Expand 10 after
826 c.Assert(err, IsNil) 826 c.Assert(err, IsNil)
827 c.Assert(needsUpgrade, DeepEquals, &state.NeedsUpgrade{true, true}) 827 c.Assert(needsUpgrade, DeepEquals, &state.NeedsUpgrade{true, true})
828 828
829 // Can be set forced multiple times. 829 // Can be set forced multiple times.
830 err = unit.SetNeedsUpgrade(true) 830 err = unit.SetNeedsUpgrade(true)
831 c.Assert(err, IsNil) 831 c.Assert(err, IsNil)
832 needsUpgrade, err = unit.NeedsUpgrade() 832 needsUpgrade, err = unit.NeedsUpgrade()
833 c.Assert(err, IsNil) 833 c.Assert(err, IsNil)
834 c.Assert(needsUpgrade, DeepEquals, &state.NeedsUpgrade{true, true}) 834 c.Assert(needsUpgrade, DeepEquals, &state.NeedsUpgrade{true, true})
835 835
836 » // Can't be set multipe with different force flag. (TODO) 836 » // Can't be set multipe with different force flag.
837 err = unit.SetNeedsUpgrade(false) 837 err = unit.SetNeedsUpgrade(false)
838 » c.Assert(err, ErrorMatches, `unit upgrade already enabled: "wordpress/0" `) 838 » c.Assert(err, ErrorMatches, `upgrade already enabled for unit "wordpress /0"`)
839 } 839 }
840 840
841 func (s *StateSuite) TestGetSetClearResolved(c *C) { 841 func (s *StateSuite) TestGetSetClearResolved(c *C) {
842 // Check that setting and clearing the resolved setting on a unit works. 842 // Check that setting and clearing the resolved setting on a unit works.
843 dummy, _ := addDummyCharm(c, s.st) 843 dummy, _ := addDummyCharm(c, s.st)
844 wordpress, err := s.st.AddService("wordpress", dummy) 844 wordpress, err := s.st.AddService("wordpress", dummy)
845 c.Assert(err, IsNil) 845 c.Assert(err, IsNil)
846 unit, err := wordpress.AddUnit() 846 unit, err := wordpress.AddUnit()
847 c.Assert(err, IsNil) 847 c.Assert(err, IsNil)
848 848
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
979 alive, err = unit.AgentAlive() 979 alive, err = unit.AgentAlive()
980 c.Assert(err, IsNil) 980 c.Assert(err, IsNil)
981 c.Assert(alive, Equals, true) 981 c.Assert(alive, Equals, true)
982 982
983 pinger.Kill() 983 pinger.Kill()
984 984
985 alive, err = unit.AgentAlive() 985 alive, err = unit.AgentAlive()
986 c.Assert(err, IsNil) 986 c.Assert(err, IsNil)
987 c.Assert(alive, Equals, false) 987 c.Assert(alive, Equals, false)
988 } 988 }
LEFTRIGHT

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b