Merge pull request #31 from nttcom-webcore/ci/auto-test

nttcom · Apr 5, 2024 · 9a750ee · 9a750ee
2 parents 1bbd38d + 2130b11
commit 9a750ee
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 16 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -14,6 +14,15 @@ jobs:
           name: Run Clippy
           command: |
             cargo clippy -- -D warnings
+  unit_test:
+    docker:
+      - image: rust:latest
+    steps:
+      - checkout
+      - run:
+          name: Run Unit Tests
+          command: |
+            cargo test
   prettier:
     docker:
       - image: cimg/node:21.6.2
@@ -34,6 +43,10 @@ workflows:
           filters:
             branches:
               ignore: /main|master/
+      - unit_test:
+          filters:
+            branches:
+              ignore: /main|master/
       - prettier:
           filters:
             branches:

diff --git a/moqt-core/src/modules/handlers/server_setup_handler.rs b/moqt-core/src/modules/handlers/server_setup_handler.rs
@@ -71,13 +71,16 @@ pub(crate) fn setup_handler(
 mod success {
     use std::vec;
 
-    use crate::modules::{
-        handlers::server_setup_handler::setup_handler,
-        messages::{
-            client_setup_message::ClientSetupMessage,
-            setup_parameters::{PathParameter, RoleCase, RoleParameter, SetupParameter},
+    use crate::{
+        constants,
+        modules::{
+            handlers::server_setup_handler::setup_handler,
+            messages::{
+                client_setup_message::ClientSetupMessage,
+                setup_parameters::{PathParameter, RoleCase, RoleParameter, SetupParameter},
+            },
+            moqt_client::MOQTClient,
         },
-        moqt_client::MOQTClient,
     };
 
     #[test]
@@ -86,7 +89,8 @@ mod success {
         let setup_parameters = vec![SetupParameter::RoleParameter(RoleParameter::new(
             RoleCase::Injection,
         ))];
-        let client_setup_message = ClientSetupMessage::new(vec![1], setup_parameters);
+        let client_setup_message =
+            ClientSetupMessage::new(vec![constants::MOQ_TRANSPORT_VERSION], setup_parameters);
         let underlay_type = crate::constants::UnderlayType::WebTransport;
 
         let server_setup_message = setup_handler(client_setup_message, underlay_type, &mut client);
@@ -102,7 +106,8 @@ mod success {
             SetupParameter::RoleParameter(RoleParameter::new(RoleCase::Injection)),
             SetupParameter::PathParameter(PathParameter::new(String::from("test"))),
         ];
-        let client_setup_message = ClientSetupMessage::new(vec![1], setup_parameters);
+        let client_setup_message =
+            ClientSetupMessage::new(vec![constants::MOQ_TRANSPORT_VERSION], setup_parameters);
         let underlay_type = crate::constants::UnderlayType::QUIC;
 
         let server_setup_message = setup_handler(client_setup_message, underlay_type, &mut client);
@@ -116,20 +121,24 @@ mod success {
 mod failure {
     use std::vec;
 
-    use crate::modules::{
-        handlers::server_setup_handler::setup_handler,
-        messages::{
-            client_setup_message::ClientSetupMessage,
-            setup_parameters::{PathParameter, SetupParameter},
+    use crate::{
+        constants,
+        modules::{
+            handlers::server_setup_handler::setup_handler,
+            messages::{
+                client_setup_message::ClientSetupMessage,
+                setup_parameters::{PathParameter, SetupParameter},
+            },
+            moqt_client::MOQTClient,
         },
-        moqt_client::MOQTClient,
     };
 
     #[test]
     fn no_setup_parameter() {
         let mut client = MOQTClient::new(33);
         let setup_parameters = vec![];
-        let client_setup_message = ClientSetupMessage::new(vec![1], setup_parameters);
+        let client_setup_message =
+            ClientSetupMessage::new(vec![constants::MOQ_TRANSPORT_VERSION], setup_parameters);
         let underlay_type = crate::constants::UnderlayType::WebTransport;
 
         let server_setup_message = setup_handler(client_setup_message, underlay_type, &mut client);
@@ -143,7 +152,8 @@ mod failure {
         let setup_parameters = vec![SetupParameter::PathParameter(PathParameter::new(
             String::from("test"),
         ))];
-        let client_setup_message = ClientSetupMessage::new(vec![1], setup_parameters);
+        let client_setup_message =
+            ClientSetupMessage::new(vec![constants::MOQ_TRANSPORT_VERSION], setup_parameters);
         let underlay_type = crate::constants::UnderlayType::WebTransport;
 
         let server_setup_message = setup_handler(client_setup_message, underlay_type, &mut client);