Cargo.toml: add crates to workspace

This allows IDE integration to work when the root of the repository is
opened. Prior to this change, a developer would have to open cargo-insta
to get code completion for that crate.

This also causes clippy to run on all crates, so this commit fixes a few
lints.

The integration-tests crate's test is changed to a proper test function
so that it can rely on always being run from the root of the
integration-tests crate. Care is taken to remove the tests directory
after running to avoid pollution in which `cargo test` attempts to run
those tests.

.github/workflows/tests.yml

@@ -24,6 +24,8 @@ jobs:
       - uses: dtolnay/rust-toolchain@master
         with:
           toolchain: 1.61.0
+      - name: Use Cargo.lock.msrv
+        run: cp Cargo.lock.msrv Cargo.lock
       - name: Test
         run: make cargotest
 

Cargo.toml

@@ -18,6 +18,9 @@ exclude = [
 all-features = true
 rustdoc-args = ["--cfg", "docsrs"]
 
+[workspace]
+members = ["cargo-insta", "cargo-insta/integration-tests"]
+
 [features]
 default = ["colors"]
 

Makefile

@@ -12,7 +12,7 @@ cargo-insta-tests:
 	@echo "CARGO-INSTA INTEGRATION TESTS"
 	# Turn off CI flag so that cargo insta test behaves as we expect
 	# under normal operation
-	@cd cargo-insta/integration-tests; CI=0 cargo run
+	@CI=0 cargo test -p integration-tests
 
 cargotest:
 	@echo "CARGO TESTS"
@@ -21,7 +21,7 @@ cargotest:
 	@cargo test --all-features
 	@cargo test --no-default-features
 	@cargo test --features redactions -- --test-threads 1
-	@cd cargo-insta; cargo test
+	@cargo test -p cargo-insta
 
 check-minver:
 	@echo "MINVER CHECK"
@@ -40,6 +40,6 @@ format-check:
 
 lint:
 	@rustup component add clippy 2> /dev/null
-	@cargo clippy
+	@cargo clippy --all-targets --workspace -- --deny warnings
 
 .PHONY: all doc test cargotest format format-check lint update-readme