test(api:repositories): test for commit listing

Author: caspiano

shard.lock

@@ -151,7 +151,7 @@ shards:
 
   placeos-build:
     git: https://github.com/placeos/build.git
-    version: 0.3.0+git.commit.0388aaece63a784c2692ed650ca471754421b99d
+    version: 0.4.0
 
   placeos-compiler:
     git: https://github.com/placeos/compiler.git

shard.yml

@@ -1,5 +1,5 @@
 name: placeos-rest-api
-version: 1.29.3
+version: 1.30.0
 crystal: ~> 1
 
 targets:
@@ -45,7 +45,7 @@ dependencies:
   # For build client
   placeos-build:
     github: placeos/build
-    branch: feat/repositories/discovery
+    version: ~> 0.4
 
   # For core client
   placeos-core:

spec/repositories_spec.cr

@@ -69,9 +69,51 @@ module PlaceOS::Api
           end
         end
 
+        describe "GET /:id/commits" do
+          context "interface" do
+            pending "fetches the commits for a repository" do
+            end
+          end
+
+          context "driver" do
+            repo = Model::Generator.repository(type: :driver)
+            repo.uri = "https://github.com/placeOS/private-drivers"
+            repo.save!
+
+            it "fetches the commits for a repository" do
+              id = repo.id.as(String)
+              path = "#{base}#{id}/commits"
+              result = curl(
+                method: "GET",
+                path: path,
+                headers: authorization_header.merge({"Content-Type" => "application/json"}),
+              )
+
+              result.status.should eq HTTP::Status::OK
+              Array(String).from_json(result.body).should_not be_empty
+            end
+
+            it "fetches the commits for a file" do
+              id = repo.id.as(String)
+              params = HTTP::Params{
+                "driver" => "drivers/place/private_helper.cr",
+              }
+              path = "#{base}#{id}/commits?#{params}"
+              result = curl(
+                method: "GET",
+                path: path,
+                headers: authorization_header.merge({"Content-Type" => "application/json"}),
+              )
+
+              result.status.should eq HTTP::Status::OK
+              Array(String).from_json(result.body).should_not be_empty
+            end
+          end
+        end
+
         describe "driver only actions" do
           it "errors if enumerating drivers in an interface repo" do
-            repository = Model::Generator.repository(type: Model::Repository::Type::Interface).save!
+            repository = Model::Generator.repository(type: :interface).save!
 
             id = repository.id.as(String)
             path = "#{base}#{id}/drivers"