powerapi 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7ca79c15ad25142ea25914fcd2d7d24f77ec5008
-  data.tar.gz: 3da80c3b78f502ef4220cdae6f78683de05ab35c
+  metadata.gz: 37841ff98ae99e8c2b0dd4b89464ab7c244a000d
+  data.tar.gz: 548bd5663509d0e6d35215ca755c5cf15eccc673
 SHA512:
-  metadata.gz: c4a88233f2b4cd8df56e2b739c5c028231957fb50b5e19c5915743f15ceb997d68722f3fad0de3f4e6f4da1ddd1dd481ee1f6f1be26e80db85c7d17dccf7e1c3
-  data.tar.gz: 3bb6dd7babbbedfa1ac0672019abcbd13892791c9b3b998bdc1bc316638d75544915ecb947f5633f3969be1e4c699cbc65bc74d8d5a67d6f6ece70ab603e6519
+  metadata.gz: c776a30b3c0bafdc48b4e758ed5a46722e2ceed47e40bf9aa6bf21159b8d2fda6c45d736ecb7a310da43c9c0d1f592020d1fff6220cd69ab6de0a0f774d565f7
+  data.tar.gz: 3586a016448f7c836d9f56db58521a1ec72357793dbdd8741b56727a89f0e13abaf380150d3d5ae124beec8ee7f4a3b2bdb504beebca111eb78b8a558858410f

data/lib/powerapi/parser.rb

@@ -81,6 +81,8 @@ module PowerAPI
         })
       end
 
+      sections = sections.sort_by{|section| [section.expression, section.name]}
+
       sections
     end
 

data/lib/powerapi/section.rb

@@ -21,6 +21,10 @@ module PowerAPI
       @details[:assignments]
     end
 
+    def expression
+      @details[:section]["expression"]
+    end
+
     def final_grades
       @final_grades
     end

data/lib/powerapi/version.rb

@@ -1,3 +1,3 @@
 module PowerAPI
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/lib/powerapi.rb

@@ -13,6 +13,8 @@ module PowerAPI
   module_function
 
   def authenticate(url, username, password, fetch_transcript=true)
+    url = clean_url(url)
+
     soap_endpoint = url + "/pearson-rest/services/PublicPortalService"
 
     login_client = Savon.client(
@@ -31,4 +33,12 @@ module PowerAPI
 
     return PowerAPI::Student.new(url, session, fetch_transcript)
   end
+
+  def clean_url(url)
+    if url[-1] == "/"
+      url = url[0..-2]
+    else
+      url = url
+    end
+  end
 end

data/spec/powerapi_spec.rb

@@ -34,4 +34,18 @@ describe PowerAPI do
       ).to be_an_instance_of(PowerAPI::Student)
     end
   end
+
+  describe "#clean_url" do
+    it "leaves 'https://powerschool.example' alone" do
+      expect(
+        PowerAPI.clean_url("https://powerschool.example")
+      ).to eq("https://powerschool.example")
+    end
+
+    it "removes the slash in 'https://powerschool.example/'" do
+      expect(
+      PowerAPI.clean_url("https://powerschool.example/")
+      ).to eq("https://powerschool.example")
+    end
+  end
 end

data/spec/section_spec.rb

@@ -57,6 +57,20 @@ describe PowerAPI::Section do
     end
   end
 
+  describe "#expression" do
+    it "has an expression of '1 (M-F)' at index 0" do
+      expect(
+        @section0.expression
+      ).to eq("1 (M-F)")
+    end
+
+    it "has an expression of '3 (M-F)' at index 1" do
+      expect(
+      @section1.expression
+      ).to eq("3 (M-F)")
+    end
+  end
+
   describe "#final_grades" do
     it "has one final grade at index 0" do
       expect(

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: powerapi
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Henri Watson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-30 00:00:00.000000000 Z
+date: 2014-12-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler