osm 0.0.26 → 0.1.0

data/spec/osm/osm_spec.rb

@@ -12,36 +12,6 @@ describe "Online Scout Manager" do
   end
 
 
-  describe "find current term ID" do
-    it "Returns the current term for the section from all terms returned by OSM" do
-      Osm::Api.configure({:api_id=>'1234', :api_token=>'12345678', :api_name=>'API', :api_site=>:scout})
-      api = Osm::Api.new('2345', 'abcd')
-
-      body = '{"9":['
-      body += '{"termid":"1","name":"Term 1","sectionid":"9","startdate":"' + (Date.today - 90).strftime('%Y-%m-%d') + '","enddate":"' + (Date.today - 31).strftime('%Y-%m-%d') + '"},'
-      body += '{"termid":"2","name":"Term 2","sectionid":"9","startdate":"' + (Date.today - 30).strftime('%Y-%m-%d') + '","enddate":"' + (Date.today + 30).strftime('%Y-%m-%d') + '"},'
-      body += '{"termid":"3","name":"Term 3","sectionid":"9","startdate":"' + (Date.today + 31).strftime('%Y-%m-%d') + '","enddate":"' + (Date.today + 90).strftime('%Y-%m-%d') + '"}'
-      body += ']}'
-      FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getTerms", :body => body)
-
-      Osm::find_current_term_id(api, 9).should == 2
-    end
-
-    it "Raises an error if there is no current term" do
-      Osm::Api.configure({:api_id=>'1234', :api_token=>'12345678', :api_name=>'API', :api_site=>:scout})
-      api = Osm::Api.new('2345', 'abcd')
-      section_id = 9
-
-      body = '{"9":['
-      body += '{"termid":"1","name":"Term 1","sectionid":"9","startdate":"' + (Date.today + 31).strftime('%Y-%m-%d') + '","enddate":"' + (Date.today + 90).strftime('%Y-%m-%d') + '"}'
-      body += ']}'
-      FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getTerms", :body => body)
-
-      expect{ Osm::find_current_term_id(api, section_id) }.to raise_error(Osm::Error)
-    end
-  end
-
-
   describe "Make a DateTime" do
     it "is given a date and a time" do
       Osm::make_datetime('2001-02-03', '04:05:06').should == DateTime.new(2001, 02, 03, 04, 05, 06)
@@ -75,8 +45,14 @@ describe "Online Scout Manager" do
     end
 
     it "is given an invalid date string" do
-      Osm::parse_date('No date here!').should == nil
+      Osm::parse_date('No date here!').should be_nil
     end
+
+    it "is given a human date" do
+      Osm::parse_date('03/02/2001').should == Date.new(2001, 02, 03)
+      Osm::parse_date('3/2/2001').should == Date.new(2001, 02, 03)
+    end
+
   end
 
 end

data/spec/osm/register_spec.rb

@@ -0,0 +1,103 @@
+# encoding: utf-8
+require 'spec_helper'
+require 'date'
+
+
+describe "Register" do
+
+  it "Create Field" do
+    field = Osm::Register::Field.new(
+      :name => 'Human name',
+      :id => 'machine_name',
+      :tooltip => 'Tooltip'
+    )
+
+    field.id.should == 'machine_name'
+    field.name.should == 'Human name'
+    field.tooltip.should == 'Tooltip'
+    field.valid?.should be_true
+  end
+
+  it "Create Data" do
+    rd = Osm::Register::Attendance.new(
+      :member_id => '1',
+      :first_name => 'A',
+      :last_name => 'B',
+      :section_id => '2',
+      :grouping_id => '3',
+      :total => 4,
+      :attendance => {
+        Date.new(2012, 1, 10) => 'Yes',
+        Date.new(2012, 1, 24) => 'No',
+      }
+    )
+
+    rd.member_id.should == 1
+    rd.section_id.should == 2
+    rd.grouping_id.should == 3
+    rd.first_name.should == 'A'
+    rd.last_name.should == 'B'
+    rd.total.should == 4
+    rd.attendance.should == {
+      Date.new(2012, 01, 10) => 'Yes',
+      Date.new(2012, 01, 24) => 'No'
+    }
+    rd.valid?.should be_true
+  end
+
+
+  describe "Using the API" do
+
+    it "Fetch the register structure for a section" do
+      data = [
+        {"rows" => [{"name"=>"First name","field"=>"firstname","width"=>"100px"},{"name"=>"Last name","field"=>"lastname","width"=>"100px"},{"name"=>"Total","field"=>"total","width"=>"60px"}],"noscroll"=>true},
+        {"rows" => []}
+      ]
+      FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/users.php?action=registerStructure&sectionid=1&termid=2", :body => data.to_json)
+
+      register_structure = Osm::Register.get_structure(@api, 1, 2)
+      register_structure.is_a?(Array).should be_true
+    end
+
+    it "Fetch the register data for a section" do
+      data = {
+        'identifier' => 'scoutid',
+        'label' => "name",
+        'items' => []
+      }
+      FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/users.php?action=register&sectionid=1&termid=2", :body => data.to_json)
+
+      register = Osm::Register.get_attendance(@api, 1, 2)
+      register.is_a?(Array).should be_true
+    end
+
+    it "Update register attendance" do
+      url = 'https://www.onlinescoutmanager.co.uk/users.php?action=registerUpdate&sectionid=1&termid=2'
+      post_data = {
+        'apiid' => @CONFIGURATION[:api][:osm][:id],
+        'token' => @CONFIGURATION[:api][:osm][:token],
+        'userid' => 'user_id',
+        'secret' => 'secret',
+        'scouts' => '["3"]',
+        'selectedDate' => '2000-01-02',
+        'present' => 'Yes',
+        'section' => :cubs,
+        'sectionid' => 1,
+        'completedBadges' => '[{"a":"A"},{"b":"B"}]'
+      }
+
+      HTTParty.should_receive(:post).with(url, {:body => post_data}) { DummyHttpResult.new(:response=>{:code=>'200', :body=>'[]'}) }
+      Osm::Register.update_attendance({
+        :api => @api,
+        :section => Osm::Section.new(:id=>1, :type=>:cubs),
+        :term => 2,
+        :evening => Date.new(2000, 1, 2),
+        :attendance => 'Yes',
+        :members => 3,
+        :completed_badge_requirements => [{'a'=>'A'}, {'b'=>'B'}]
+      }).should be_true
+    end
+
+  end
+
+end

data/spec/osm/section_spec.rb

@@ -5,83 +5,46 @@ require 'date'
 
 describe "Section" do
 
-  class DummyRole
-    attr_reader :id
-    def initialize(id)
-      @id = id
-    end
-    def <=>(another)
-      @id <=> another.id
-    end
-  end
-
-
   before :each do
     @attributes = {
       :id => 1,
       :name => 'Name',
       :subscription_level => :silver,
       :subscription_expires => (Date.today + 60).strftime('%Y-%m-%d'),
-      :sectionType => :cubs,
-      :numscouts => 10,
-      :hasUsedBadgeRecords => true,
-      :hasProgramme => true,
+      :type => :cubs,
+      :num_scouts => 10,
       :wizard => false,
-      :columnNames => {},
-      :fields => {},
-      :intouch => {},
-      :mobFields => {},
-      :extraRecords => [],
-      :role => DummyRole.new(1)
+      :column_names => {:column_names => 'names'},
+      :fields => {:fields => true},
+      :intouch_fields => {:intouch_fields => true},
+      :mobile_fields => {:mobile_fields => true},
+      :flexi_records => [],
+      :group_id => 3,
+      :group_name => '3rd Somewhere'
     }
   end
 
 
-  it "Create from API data" do
-    data = {
-      'subscription_level' => '3',
-      'subscription_expires' => (Date.today + 60).strftime('%Y-%m-%d'),
-      'sectionType' => 'cubs',
-      'numscouts' => 10,
-      'columnNames' => {},
-      'fields' => {},
-      'intouch' => {},
-      'mobFields' => {},
-      'extraRecords' => [
-        {
-          'name' => 'Name 1',
-          'extraid' => 1
-        }, [
-          '',
-          {
-            'name' => 'Name 2',
-            'extraid' => 2
-          }
-        ]
-      ]
-    }
-    role = DummyRole.new(1)
-    section = Osm::Section.from_api(1, 'Name', data, role)
+  it "Create" do
+    section = Osm::Section.new(@attributes)
 
     section.id.should == 1
     section.name.should == 'Name' 
-    section.subscription_level.should == :gold
+    section.subscription_level.should == :silver
     section.subscription_expires.should == Date.today + 60
     section.type.should == :cubs
     section.num_scouts.should == 10
-    section.column_names.should == {}
-    section.fields.should == {}
-    section.intouch_fields.should == {}
-    section.mobile_fields.should == {}
-    section.flexi_records[0].id.should == 1
-    section.flexi_records[0].name.should == 'Name 1'
-    section.flexi_records[1].id.should == 2
-    section.flexi_records[1].name.should == 'Name 2'
-    section.role.should == role
-
+    section.column_names.should == {:column_names => 'names'}
+    section.fields.should == {:fields => true}
+    section.intouch_fields.should == {:intouch_fields => true}
+    section.mobile_fields.should == {:mobile_fields => true}
+    section.group_id.should == 3
+    section.group_name.should == '3rd Somewhere'
+    section.flexi_records.should == []
     section.valid?.should be_true
   end
 
+
   it "Create has sensible defaults" do
     section = Osm::Section.new
 
@@ -97,67 +60,157 @@ describe "Section" do
   end
 
 
-  it "Compares two matching sections" do
-    section1 = Osm::Section.new(@attributes)
-    section2 = section1.clone
+  describe "Using the API" do
+
+    before :each do
+      body = [
+        {"sectionConfig"=>"{\"subscription_level\":1,\"subscription_expires\":\"2013-01-05\",\"sectionType\":\"beavers\",\"columnNames\":{\"column_names\":\"names\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":[{\"name\":\"Flexi Record 1\",\"extraid\":\"111\"}],\"wizard\":\"false\",\"fields\":{\"fields\":true},\"intouch\":{\"intouch_fields\":true},\"mobFields\":{\"mobile_fields\":true}}", "groupname"=>"3rd Somewhere", "groupid"=>"3", "groupNormalised"=>"1", "sectionid"=>"1", "sectionname"=>"Section 1", "section"=>"beavers", "isDefault"=>"1", "permissions"=>{"badge"=>10, "member"=>20, "user"=>100, "register"=>100, "contact"=>100, "programme"=>100, "originator"=>1, "events"=>100, "finance"=>100, "flexi"=>100}},
+        {"sectionConfig"=>"{\"subscription_level\":3,\"subscription_expires\":\"2013-01-05\",\"sectionType\":\"cubs\",\"columnNames\":{\"phone1\":\"Home Phone\",\"phone2\":\"Parent 1 Phone\",\"address\":\"Member's Address\",\"phone3\":\"Parent 2 Phone\",\"address2\":\"Address 2\",\"phone4\":\"Alternate Contact Phone\",\"subs\":\"Gender\",\"email1\":\"Parent 1 Email\",\"medical\":\"Medical / Dietary\",\"email2\":\"Parent 2 Email\",\"ethnicity\":\"Gift Aid\",\"email3\":\"Member's Email\",\"religion\":\"Religion\",\"email4\":\"Email 4\",\"school\":\"School\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":[],\"wizard\":\"false\",\"fields\":{\"email1\":true,\"email2\":true,\"email3\":true,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":true,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":true,\"saved\":true},\"intouch\":{\"address\":true,\"address2\":false,\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"medical\":false},\"mobFields\":{\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":false,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":false}}", "groupname"=>"1st Somewhere", "groupid"=>"1", "groupNormalised"=>"1", "sectionid"=>"2", "sectionname"=>"Section 2", "section"=>"cubs", "isDefault"=>"0", "permissions"=>{"badge"=>100, "member"=>100, "user"=>100, "register"=>100, "contact"=>100, "programme"=>100, "originator"=>1, "events"=>100, "finance"=>100, "flexi"=>100}}
+      ]
+      FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getUserRoles", :body => body.to_json)
+    end
+
+    it "Gets all sections" do
+      sections = Osm::Section.get_all(@api)
+      sections.map{ |i| i.id }.should == [1, 2]
+
+      section = sections[0]
+      section.id.should == 1
+      section.name.should == 'Section 1' 
+      section.subscription_level.should == :bronze
+      section.subscription_expires.should == Date.new(2013, 1, 5)
+      section.type.should == :beavers
+      section.num_scouts.should == 10
+      section.column_names.should == {:column_names => 'names'}
+      section.fields.should == {:fields => true}
+      section.intouch_fields.should == {:intouch_fields => true}
+      section.mobile_fields.should == {:mobile_fields => true}
+      section.group_id.should == 3
+      section.group_name.should == '3rd Somewhere'
+      section.flexi_records.size.should == 1
+      section.flexi_records[0].id.should == 111
+      section.flexi_records[0].name.should == 'Flexi Record 1'
+    end
+  
+    it "Gets a section" do
+      section = Osm::Section.get(@api, 1)
+      section.should_not be_nil
+      section.id.should == 1
+    end
+
+    it "Fetches user's permissions" do
+      permissions = Osm::Section.fetch_user_permissions(@api)
+      permissions[1].should_not be_nil
+      permissions[1].should == {
+        :badge => [:read],
+        :member => [:read, :write],
+        :user => [:read, :write, :administer],
+        :register => [:read, :write, :administer],
+        :contact => [:read, :write, :administer],
+        :programme => [:read, :write, :administer],
+        :originator => [],
+        :events => [:read, :write, :administer],
+        :finance => [:read, :write, :administer],
+        :flexi => [:read, :write, :administer],
+      }
+    end
+
+    it "Gets the section's notepad" do
+      FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getNotepads", :body => {"1" => "Section 1", "2" => "Section 2"}.to_json)
+      section = Osm::Section.get(@api, 1)
+      section.should_not be_nil
+      section.get_notepad(@api).should == 'Section 1'
+    end
 
-    section1.should == section2
   end
 
-  it "Compares two non-matching sections" do
-    section1 = Osm::Section.new(@attributes)
-    section2 = Osm::Section.new(@attributes.merge(:id => 2))
 
-    section1.should_not == section2
+  describe "Compare two sections" do
+
+    it "They match" do
+      section1 = Osm::Section.new(@attributes)
+      section2 = Osm::Section.new(@attributes)
+
+      section1.should == section2
+    end
+
+    it "They don't match" do
+      section1 = Osm::Section.new(@attributes)
+      section2 = Osm::Section.new(@attributes.merge(:id => 2))
+
+      section1.should_not == section2
+    end
+
   end
 
 
-  it "Sorts by role" do
-    section1 = Osm::Section.new(@attributes.merge(:role => DummyRole.new(1)))
-    section2 = Osm::Section.new(@attributes.merge(:role => DummyRole.new(2)))
+  it "Sorts by Group Name then section type (age order)" do
+    section1 = Osm::Section.new(@attributes.merge(:group_id => 1, :group_name => '1st Somewhere', :type => :beavers))
+    section2 = Osm::Section.new(@attributes.merge(:group_id => 2, :group_name => '2nd Somewhere', :type => :beavers))
+    section3 = Osm::Section.new(@attributes.merge(:group_id => 2, :group_name => '2nd Somewhere', :type => :cubs))
 
-    [section2, section1].sort.should == [section1, section2]
+    [section2, section3, section1].sort.should == [section1, section2, section3]
   end
 
 
-  it "Correctly works out the section type" do
-    unknown =   Osm::Section.new(@attributes.merge(:id => 1, :role => DummyRole.new(1)))
-    beavers =   Osm::Section.new(@attributes.merge(:id => 2, :type => :beavers, :role => DummyRole.new(2)))
-    cubs =      Osm::Section.new(@attributes.merge(:id => 3, :type => :cubs, :role => DummyRole.new(3)))
-    scouts =    Osm::Section.new(@attributes.merge(:id => 4, :type => :scouts, :role => DummyRole.new(4)))
-    explorers = Osm::Section.new(@attributes.merge(:id => 5, :type => :explorers, :role => DummyRole.new(5)))
-    adults =    Osm::Section.new(@attributes.merge(:id => 6, :type => :adults, :role => DummyRole.new(6)))
-    waiting =   Osm::Section.new(@attributes.merge(:id => 7, :type => :waiting, :role => DummyRole.new(7)))
+  describe "Correctly works out the section type" do
+    unknown   = Osm::Section.new(:type => :abc)
+    beavers   = Osm::Section.new(:type => :beavers)
+    cubs      = Osm::Section.new(:type => :cubs)
+    scouts    = Osm::Section.new(:type => :scouts)
+    explorers = Osm::Section.new(:type => :explorers)
+    adults    = Osm::Section.new(:type => :adults)
+    waiting   = Osm::Section.new(:type => :waiting)
 
-    {:beavers => beavers, :cubs => cubs, :scouts => scouts, :explorers => explorers, :adults => adults, :waiting => waiting, :unknwoon => unknown}.each do |section_type, section|
-      [:beavers, :cubs, :scouts, :explorers, :adults, :waiting].each do |type|
-        section.send("#{type.to_s}?").should == (section_type == type)
+    {:beavers => beavers, :cubs => cubs, :scouts => scouts, :explorers => explorers, :adults => adults, :waiting => waiting, :unknown => unknown}.each do |section_type, section|
+      it "For a #{section_type} section" do
+        [:beavers, :cubs, :scouts, :explorers, :adults, :waiting].each do |type|
+          section.send("#{type.to_s}?").should == (section_type == type)
+        end
       end
     end
   end
 
 
-  it "Correctly works out if the section is a youth section" do
-    unknown =   Osm::Section.new(@attributes.merge(:id => 1, :role => DummyRole.new(1)))
-    beavers =   Osm::Section.new(@attributes.merge(:id => 2, :type => :beavers, :role => DummyRole.new(2)))
-    cubs =      Osm::Section.new(@attributes.merge(:id => 3, :type => :cubs, :role => DummyRole.new(3)))
-    scouts =    Osm::Section.new(@attributes.merge(:id => 4, :type => :scouts, :role => DummyRole.new(4)))
-    explorers = Osm::Section.new(@attributes.merge(:id => 5, :type => :explorers, :role => DummyRole.new(5)))
-    adults =    Osm::Section.new(@attributes.merge(:id => 6, :type => :adults, :role => DummyRole.new(6)))
-    waiting =   Osm::Section.new(@attributes.merge(:id => 7, :type => :waiting, :role => DummyRole.new(7)))
+  describe "Correctly works out if the section is a youth section" do
+    unknown =   Osm::Section.new(:type => :abc)
+    beavers =   Osm::Section.new(:type => :beavers)
+    cubs =      Osm::Section.new(:type => :cubs)
+    scouts =    Osm::Section.new(:type => :scouts)
+    explorers = Osm::Section.new(:type => :explorers)
+    adults =    Osm::Section.new(:type => :adults)
+    waiting =   Osm::Section.new(:type => :waiting)
 
     [beavers, cubs, scouts, explorers].each do |section|
-      section.youth_section?.should == true
+      it "For a #{section.type} section" do
+        section.youth_section?.should be_true
+      end
     end
     [adults, waiting, unknown].each do |section|
-      section.youth_section?.should == false
+      it "For a #{section.type} section" do
+        section.youth_section?.should be_false
+      end
     end
   end
 
+end
+
+
+
+describe "Section::FlexiRecord" do
+
+  it "Create" do
+    fr = Osm::Section::FlexiRecord.new(:id => 1, :name => 'Name')
+
+    fr.id.should == 1
+    fr.name.should == 'Name' 
+    fr.valid?.should be_true
+  end
+
 
   it "Sorts Flexi Records by name" do
-    fr1 = Osm::Section::FlexiRecord.new(:id => 1, :name => 'A')
-    fr2 = Osm::Section::FlexiRecord.new(:id => 1, :name => 'B')
+    fr1 = Osm::Section::FlexiRecord.new(:id => 3, :name => 'A')
+    fr2 = Osm::Section::FlexiRecord.new(:id => 2, :name => 'B')
     fr3 = Osm::Section::FlexiRecord.new(:id => 1, :name => 'C')
     records = [fr2, fr1, fr3]
 
@@ -165,3 +218,66 @@ describe "Section" do
   end
 
 end
+
+
+
+describe "Online Scout Manager API Strangeness" do
+
+  it "handles a section with no type" do
+    body = '[{"sectionConfig":"{\"subscription_level\":3,\"subscription_expires\":\"2013-01-05\",\"columnNames\":{\"phone1\":\"Home Phone\",\"phone2\":\"Parent 1 Phone\",\"address\":\"Member\'s Address\",\"phone3\":\"Parent 2 Phone\",\"address2\":\"Address 2\",\"phone4\":\"Alternate Contact Phone\",\"subs\":\"Gender\",\"email1\":\"Parent 1 Email\",\"medical\":\"Medical / Dietary\",\"email2\":\"Parent 2 Email\",\"ethnicity\":\"Gift Aid\",\"email3\":\"Member\'s Email\",\"religion\":\"Religion\",\"email4\":\"Email 4\",\"school\":\"School\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":[{\"name\":\"Subs\",\"extraid\":\"529\"}],\"wizard\":\"false\",\"fields\":{\"email1\":true,\"email2\":true,\"email3\":true,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":true,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":true,\"saved\":true},\"intouch\":{\"address\":true,\"address2\":false,\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"medical\":false},\"mobFields\":{\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":false,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":false}}","groupname":"1st Somewhere","groupid":"1","groupNormalised":"1","sectionid":"1","sectionname":"Section 1","section":"cubs","isDefault":"1","permissions":{"badge":100,"member":100,"user":100,"register":100,"contact":100,"programme":100,"originator":1,"events":100,"finance":100,"flexi":100}}]'
+    FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getUserRoles", :body => body)
+
+    sections = Osm::Section.get_all(@api)
+    sections.size.should == 1
+    section = sections[0]
+    section.should_not be_nil
+    section.type.should == :unknown
+  end
+
+  it "handles strange extra records when getting roles" do
+    body = '[{"sectionConfig":"{\"subscription_level\":3,\"subscription_expires\":\"2013-01-05\",\"sectionType\":\"cubs\",\"columnNames\":{\"phone1\":\"Home Phone\",\"phone2\":\"Parent 1 Phone\",\"address\":\"Member\'s Address\",\"phone3\":\"Parent 2 Phone\",\"address2\":\"Address 2\",\"phone4\":\"Alternate Contact Phone\",\"subs\":\"Gender\",\"email1\":\"Parent 1 Email\",\"medical\":\"Medical / Dietary\",\"email2\":\"Parent 2 Email\",\"ethnicity\":\"Gift Aid\",\"email3\":\"Member\'s Email\",\"religion\":\"Religion\",\"email4\":\"Email 4\",\"school\":\"School\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":[[\"1\",{\"name\":\"Subs\",\"extraid\":\"529\"}],[\"2\",{\"name\":\"Subs 2\",\"extraid\":\"530\"}]],\"wizard\":\"false\",\"fields\":{\"email1\":true,\"email2\":true,\"email3\":true,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":true,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":true,\"saved\":true},\"intouch\":{\"address\":true,\"address2\":false,\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"medical\":false},\"mobFields\":{\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":false,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":false}}","groupname":"1st Somewhere","groupid":"1","groupNormalised":"1","sectionid":"1","sectionname":"Section 1","section":"cubs","isDefault":"1","permissions":{"badge":100,"member":100,"user":100,"register":100,"contact":100,"programme":100,"originator":1,"events":100,"finance":100,"flexi":100}}]'
+    FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getUserRoles", :body => body)
+
+    sections = Osm::Section.get_all(@api)
+    sections.size.should == 1
+    sections[0].should_not be_nil
+  end
+
+  it "handles a section config where fields is an empty array" do
+    body = '[{"sectionConfig":"{\"subscription_level\":3,\"subscription_expires\":\"2013-01-05\",\"columnNames\":{\"phone1\":\"Home Phone\",\"phone2\":\"Parent 1 Phone\",\"address\":\"Member\'s Address\",\"phone3\":\"Parent 2 Phone\",\"address2\":\"Address 2\",\"phone4\":\"Alternate Contact Phone\",\"subs\":\"Gender\",\"email1\":\"Parent 1 Email\",\"medical\":\"Medical / Dietary\",\"email2\":\"Parent 2 Email\",\"ethnicity\":\"Gift Aid\",\"email3\":\"Member\'s Email\",\"religion\":\"Religion\",\"email4\":\"Email 4\",\"school\":\"School\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":[{\"name\":\"Subs\",\"extraid\":\"529\"}],\"wizard\":\"false\",\"fields\":[],\"intouch\":{\"address\":true,\"address2\":false,\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"medical\":false},\"mobFields\":{\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":false,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":false}}","groupname":"1st Somewhere","groupid":"1","groupNormalised":"1","sectionid":"1","sectionname":"Section 1","section":"cubs","isDefault":"1","permissions":{"badge":100,"member":100,"user":100,"register":100,"contact":100,"programme":100,"originator":1,"events":100,"finance":100,"flexi":100}}]'
+    FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getUserRoles", :body => body)
+
+    sections = Osm::Section.get_all(@api)
+    sections.size.should == 1
+    section = sections[0]
+    section.should_not be_nil
+    section.fields.should == {}
+  end
+
+  it "handles a section's flexi records being a hash" do
+    body = [
+      {"sectionConfig"=>"{\"subscription_level\":1,\"subscription_expires\":\"2013-01-05\",\"sectionType\":\"beavers\",\"columnNames\":{\"phone1\":\"Home Phone\",\"phone2\":\"Parent 1 Phone\",\"address\":\"Member's Address\",\"phone3\":\"Parent 2 Phone\",\"address2\":\"Address 2\",\"phone4\":\"Alternate Contact Phone\",\"subs\":\"Gender\",\"email1\":\"Parent 1 Email\",\"medical\":\"Medical / Dietary\",\"email2\":\"Parent 2 Email\",\"ethnicity\":\"Gift Aid\",\"email3\":\"Member's Email\",\"religion\":\"Religion\",\"email4\":\"Email 4\",\"school\":\"School\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":{\"1\":{\"name\":\"Flexi Record 1\",\"extraid\":\"1\"},\"2\":{\"name\":\"Flexi Record 2\",\"extraid\":\"2\"}},\"wizard\":\"false\",\"fields\":{\"email1\":true,\"email2\":true,\"email3\":true,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":true,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":true,\"saved\":true},\"intouch\":{\"address\":true,\"address2\":false,\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"medical\":false},\"mobFields\":{\"email1\":false,\"email2\":false,\"email3\":false,\"email4\":false,\"address\":true,\"address2\":false,\"phone1\":true,\"phone2\":true,\"phone3\":true,\"phone4\":true,\"school\":false,\"religion\":false,\"ethnicity\":true,\"medical\":true,\"patrol\":true,\"subs\":false}}", "groupname"=>"3rd Somewhere", "groupid"=>"3", "groupNormalised"=>"1", "sectionid"=>"1", "sectionname"=>"Section 1", "section"=>"beavers", "isDefault"=>"1", "permissions"=>{"badge"=>100, "member"=>100, "user"=>100, "register"=>100, "contact"=>100, "programme"=>100, "originator"=>1, "events"=>100, "finance"=>100, "flexi"=>100}},
+    ]
+    FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getUserRoles", :body => body.to_json)
+
+    sections = Osm::Section.get_all(@api)
+    sections.size.should == 1
+    section = sections[0]
+    section.should_not be_nil
+    section.flexi_records.size.should == 2
+    fr = section.flexi_records[0]
+    fr.id.should == 1
+    fr.name.should == 'Flexi Record 1'
+  end
+
+  it "handles an empty array representing no notepads" do
+    body = [{"sectionConfig"=>"{\"subscription_level\":1,\"subscription_expires\":\"2013-01-05\",\"sectionType\":\"beavers\",\"columnNames\":{\"column_names\":\"names\"},\"numscouts\":10,\"hasUsedBadgeRecords\":true,\"hasProgramme\":true,\"extraRecords\":[{\"name\":\"Flexi Record 1\",\"extraid\":\"111\"}],\"wizard\":\"false\",\"fields\":{\"fields\":true},\"intouch\":{\"intouch_fields\":true},\"mobFields\":{\"mobile_fields\":true}}", "groupname"=>"3rd Somewhere", "groupid"=>"3", "groupNormalised"=>"1", "sectionid"=>"1", "sectionname"=>"Section 1", "section"=>"beavers", "isDefault"=>"1", "permissions"=>{"badge"=>10, "member"=>20, "user"=>100, "register"=>100, "contact"=>100, "programme"=>100, "originator"=>1, "events"=>100, "finance"=>100, "flexi"=>100}}]
+    FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getUserRoles", :body => body.to_json)
+    FakeWeb.register_uri(:post, "https://www.onlinescoutmanager.co.uk/api.php?action=getNotepads", :body => '[]')
+
+    section = Osm::Section.get(@api, 1)
+    section.should_not be_nil
+    section.get_notepad(@api).should == ''
+  end
+
+end