Merge pull request #95 from hug-dev/use-parsec-mock

Use Parsec Mock for testing

Author: ionut-arm

.github/workflows/ci.yml

@@ -7,6 +7,16 @@ jobs:
     name: Build and check formatting
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
-      - name: Execute all tests
-        run: ./tests/ci.sh
+      - uses: actions/checkout@v2
+      - name: Set up Python 3.7
+        uses: actions/setup-python@v1
+        with:
+          python-version: 3.7
+      - name: Download Parsec Mock, install dependencies and execute all tests
+        run: |
+          curl -s -N -L https://github.com/parallaxsecond/parsec-mock/archive/refs/tags/0.1.1.tar.gz | tar xz
+          cd parsec-mock-0.1.1/
+          python -m pip install --upgrade pip
+          pip install -r requirements.txt
+          cd ..
+          ./tests/ci.sh

src/core/basic_client.rs

@@ -116,16 +116,16 @@ impl BasicClient {
     ///#
     ///# fn main() -> Result<(), Box<dyn Error>> {
     ///use parsec_client::BasicClient;
-    ///let client = BasicClient::new_naked();
+    ///let client = BasicClient::new_naked()?;
     ///let (major, minor) = client.ping()?;
     ///# Ok(())}
     /// ```
-    pub fn new_naked() -> Self {
-        BasicClient {
-            op_client: Default::default(),
+    pub fn new_naked() -> Result<Self> {
+        Ok(BasicClient {
+            op_client: OperationClient::new()?,
             auth_data: Authentication::None,
             implicit_provider: ProviderId::Core,
-        }
+        })
     }
 
     /// Query the service for the list of authenticators provided and use the first one as default
@@ -150,7 +150,7 @@ impl BasicClient {
     ///# fn main() -> Result<(), Box<dyn Error>> {
     ///use parsec_client::BasicClient;
     ///use parsec_client::core::interface::requests::ProviderId;
-    ///let mut client = BasicClient::new_naked();
+    ///let mut client = BasicClient::new_naked()?;
     ///// Set the default authenticator but choose a specific provider.
     ///client.set_implicit_provider(ProviderId::Pkcs11);
     ///client.set_default_auth(Some("main_client".to_string()))?;
@@ -209,7 +209,7 @@ impl BasicClient {
     ///# fn main() -> Result<(), Box<dyn Error>> {
     ///use parsec_client::BasicClient;
     ///use parsec_client::auth::Authentication;
-    ///let mut client = BasicClient::new_naked();
+    ///let mut client = BasicClient::new_naked()?;
     ///client.set_auth_data(Authentication::UnixPeerCredentials);
     ///assert_eq!(Authentication::UnixPeerCredentials, client.auth_data());
     ///# Ok(())}
@@ -233,7 +233,7 @@ impl BasicClient {
     ///# fn main() -> Result<(), Box<dyn Error>> {
     ///use parsec_client::BasicClient;
     ///use parsec_client::auth::Authentication;
-    ///let mut client = BasicClient::new_naked();
+    ///let mut client = BasicClient::new_naked()?;
     ///// Use the default provider but use a specific authentication.
     ///client.set_default_provider()?;
     ///client.set_auth_data(Authentication::UnixPeerCredentials);
@@ -268,7 +268,7 @@ impl BasicClient {
     ///# fn main() -> Result<(), Box<dyn Error>> {
     ///use parsec_client::BasicClient;
     ///use parsec_client::core::interface::requests::ProviderId;
-    ///let mut client = BasicClient::new_naked();
+    ///let mut client = BasicClient::new_naked()?;
     ///client.set_implicit_provider(ProviderId::Pkcs11);
     ///assert_eq!(ProviderId::Pkcs11, client.implicit_provider());
     ///# Ok(())}
@@ -327,7 +327,7 @@ impl BasicClient {
     ///# fn main() -> Result<(), Box<dyn Error>> {
     ///use parsec_client::BasicClient;
     ///
-    ///let mut client: BasicClient = BasicClient::new_naked();
+    ///let mut client: BasicClient = BasicClient::new_naked()?;
     ///let providers = client.list_providers()?;
     ///// Set the second most prioritary provider
     ///client.set_implicit_provider(providers[1].id);
@@ -1291,3 +1291,13 @@ impl BasicClient {
         }
     }
 }
+
+impl Default for BasicClient {
+    fn default() -> Self {
+        BasicClient {
+            op_client: Default::default(),
+            auth_data: Authentication::None,
+            implicit_provider: ProviderId::Core,
+        }
+    }
+}

src/core/testing/core_tests.rs

@@ -56,22 +56,6 @@ fn get_operation_from_req_bytes(bytes: Vec<u8>) -> NativeOperation {
         .unwrap()
 }
 
-#[test]
-fn ping_test() {
-    let mut client: TestBasicClient = Default::default();
-    client.set_mock_read(&get_response_bytes_from_result(NativeResult::Ping(
-        operations::ping::Result {
-            wire_protocol_version_maj: 1,
-            wire_protocol_version_min: 0,
-        },
-    )));
-    // Check request:
-    // Ping request is empty so no checking to be done
-
-    // Check response:
-    assert_eq!(client.ping().expect("Ping failed"), (1, 0));
-}
-
 #[test]
 fn list_provider_test() {
     let mut client: TestBasicClient = Default::default();

tests/ci.sh

@@ -27,6 +27,18 @@ if cargo clippy -h; then
 	cargo clippy --all-targets -- -D clippy::all -D clippy::cargo
 fi
 
+######################
+# Start Mock Service #
+######################
+CURRENT_PATH=$(pwd)
+cd parsec-mock-0.1.1
+python parsec_mock/parsec_mock.py --parsec-socket $CURRENT_PATH/parsec_mock.sock &
+while [[ ! -S $CURRENT_PATH/parsec_mock.sock ]]; do
+	sleep 5
+done
+cd ..
+export PARSEC_SERVICE_ENDPOINT="unix://$CURRENT_PATH/parsec_mock.sock"
+
 #############
 # Run tests #
 #############

tests/mock_service.rs

@@ -0,0 +1,11 @@
+// Copyright 2021 Contributors to the Parsec project.
+// SPDX-License-Identifier: Apache-2.0
+use parsec_client::BasicClient;
+
+#[test]
+fn ping_noauth() {
+    let client = BasicClient::new_naked().unwrap();
+
+    // ping_noauth request
+    assert_eq!(client.ping().expect("Ping failed"), (1, 0));
+}