Rename integration tests in system tests

.github/workflows/integration.yml → .github/workflows/system-tests.yml

@@ -1,4 +1,4 @@
-name: Integration tests
+name: System tests
 on:
   push:
     branches:
@@ -9,7 +9,7 @@ on:
     paths-ignore:
       - 'docs/**'
 jobs:
-  integration:
+  system:
     runs-on: ubuntu-22.04
     services:
       mailhog:
@@ -54,4 +54,4 @@ jobs:
           go install
       - name: Test
         run: |
-          make integration-tests
+          make system-tests

Makefile

@@ -30,7 +30,7 @@ scripts/golangci-lint: Makefile
 
 ## jslint: enforce a consistent code style for Js code
 jslint: scripts/node_modules
-	@scripts/node_modules/.bin/eslint -c scripts/eslintrc.json "assets/scripts/**" tests/integration/konnector/*.js
+	@scripts/node_modules/.bin/eslint -c scripts/eslintrc.json "assets/scripts/**" tests/system/konnector/*.js
 .PHONY: jslint
 
 ## pretty: make the assets prettier
@@ -74,10 +74,10 @@ unit-tests:
 	@go test -p 1 -timeout 2m -short ./...
 .PHONY: unit-tests
 
-## integration-tests: run the tests
-integration-tests:
-	@scripts/integration-test.sh
-.PHONY: integration-tests
+## system-tests: run the tests
+system-tests:
+	@scripts/system-test.sh
+.PHONY: system-tests
 
 ## clean: clean the generated files and directories
 clean:

cmd/tools.go

@@ -90,10 +90,10 @@ var encryptRSACmd = &cobra.Command{
 	Use:   "encrypt-with-rsa <key> <payload",
 	Short: "encrypt a payload in RSA",
 	Long: `
-This command is used by integration tests to encrypt bitwarden organization
-keys. It takes the public or private key of the user and the payload (= the
-organization key) as inputs (both encoded in base64), and print on stdout the
-encrypted data (encoded as base64 too).
+This command is used by system tests to encrypt bitwarden organization keys. It
+takes the public or private key of the user and the payload (= the organization
+key) as inputs (both encoded in base64), and print on stdout the encrypted data
+(encoded as base64 too).
 `,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if len(args) != 2 {

docs/cli/cozy-stack_tools_encrypt-with-rsa.md

@@ -5,10 +5,10 @@ encrypt a payload in RSA
 ### Synopsis
 
 
-This command is used by integration tests to encrypt bitwarden organization
-keys. It takes the public or private key of the user and the payload (= the
-organization key) as inputs (both encoded in base64), and print on stdout the
-encrypted data (encoded as base64 too).
+This command is used by system tests to encrypt bitwarden organization keys. It
+takes the public or private key of the user and the payload (= the organization
+key) as inputs (both encoded in base64), and print on stdout the encrypted data
+(encoded as base64 too).
 
 
 ```

model/job/mem_scheduler.go

@@ -53,11 +53,11 @@ func (s *memScheduler) StartScheduler(b Broker) error {
 	go s.thumb.Schedule()
 
 	// XXX The memory scheduler loads the triggers from CouchDB when the stack
-	// is started. This can cause some stability issues when running
-	// integration tests in parallel. To avoid that, an env variable
+	// is started. This can cause some stability issues when running system
+	// tests in parallel. To avoid that, an env variable
 	// COZY_SKIP_LOADING_TRIGGERS can be set to skip loading the triggers from
-	// CouchDB. It is correct for integration tests, as instances are created
-	// and destroyed by the same process. But, it should not be used elsewhere.
+	// CouchDB. It is correct for system tests, as instances are created and
+	// destroyed by the same process. But, it should not be used elsewhere.
 	for _, env := range os.Environ() {
 		if strings.HasPrefix(env, "COZY_SKIP_LOADING_TRIGGERS=") {
 			return nil

scripts/integration-test.sh → scripts/system-test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 set -ev
 
-cd tests/integration
+cd tests/system
 sudo npm install -g @bitwarden/cli@1.16.0
 
 bundle install --jobs=3 --retry=3

tests/integration/README.md → tests/system/README.md

@@ -1,4 +1,4 @@
-# Tools for integration tests
+# Tools for system tests
 
 ## Install
 
@@ -7,7 +7,7 @@ apt install ruby ruby-dev
 gem install bundler
 cd cozy-stack
 go install
-cd tests/integration
+cd tests/system
 bundle install
 npm install -g @bitwarden/cli@1.16.0
 ```
@@ -74,7 +74,7 @@ it with swifttest:
 
 ```sh
 $ go run ./tests/swift
-$ cd tests/integration
+$ cd tests/system
 $ export COZY_SWIFTTEST=1
 $ bundle exec ruby tests/sharing_push_folder.rb
 ```

tests/integration/hammer-runner.sh → tests/system/hammer-runner.sh

@@ -2,8 +2,8 @@
 set -e
 
 # Usage: ./hammer-runner.sh tests/accounts_cleaning.rb
-# It will launch the same integration tests in a loop until it fails (up to 100
-# tries). It is useful for trying to trigger an error on an integration test
+# It will launch the same system tests in a loop until it fails (up to 100
+# tries). It is useful for trying to trigger an error on an system test
 # that fails on some conditions that happen sometimes, but not very often.
 
 bundle exec ruby clean.rb

tests/integration/lib/oauth/client.rb → tests/system/lib/oauth/client.rb

@@ -6,7 +6,7 @@ def self.create(inst)
       body = {
         redirect_uris: ["cozy://"],
         client_name: "test_#{Faker::Internet.slug}",
-        software_id: "github.com/cozy/cozy-stack/tests/integration"
+        software_id: "github.com/cozy/cozy-stack/tests/system"
       }
       opts = { content_type: :json, accept: :json }
       res = inst.client["/auth/register"].post body.to_json, opts

tests/integration/lib/stack.rb → tests/system/lib/stack.rb

@@ -156,7 +156,7 @@ def generate_token_for(inst, doctypes)
   def generate_client_id(inst)
     cmd = ["cozy-stack", "instances", "client-oauth", inst.domain,
            "--admin-port", @admin,
-           "http://localhost", "test-sharing", "github.com/cozy/cozy-stack/tests/integration"]
+           "http://localhost", "test-sharing", "github.com/cozy/cozy-stack/tests/system"]
     puts cmd.join(" ").green
     `#{cmd.join(" ")}`.chomp
   end