refactor: collapse assertions to speed up tests

1 files changed, 46 insertions, 36 deletions

diff --git a/spec/integration/java/gradle_spec.rb b/spec/integration/java/gradle_spec.rb
index e6fbc1d..dd0ba26 100644
--- a/spec/integration/java/gradle_spec.rb
+++ b/spec/integration/java/gradle_spec.rb
@@ -3,41 +3,40 @@
 require 'spec_helper'
 
 RSpec.describe "gradle" do
+  subject { runner.scan(env: env) }
+
+  let(:env) { {} }
+
   include_examples "each report version", "java", "gradle"
 
   context "when running a default gradle build" do
-    it 'scans a gradle project' do
-      content = <<~GRADLE
-/*
- * This file was generated by the Gradle 'init' task.
- *
- * This is a general purpose Gradle build.
- * Learn how to create Gradle builds at https://guides.gradle.org/creating-new-gradle-builds
- */
+    before do
+      runner.add_file('build.gradle') do
+        <<~GRADLE
 plugins {
   id "com.github.hierynomus.license" version "0.15.0"
 }
-      GRADLE
-      runner.add_file('build.gradle', content)
+        GRADLE
+      end
+    end
 
-      report = runner.scan
-      expect(report).to match_schema
-      expect(report[:licenses]).to be_empty
-      expect(report[:dependencies]).to be_empty
+    it 'scans a gradle project' do
+      expect(subject).to match_schema
+      expect(subject[:licenses]).to be_empty
+      expect(subject[:dependencies]).to be_empty
     end
   end
 
   context 'when scanning a gradle project that does not include the `com.github.hierynomus.license` plugin' do
     let(:project_url) { 'https://gitlab.com/one-touch-pipeline/otp.git' }
-    let(:result) { runner.scan }
 
     before do
       runner.clone(project_url)
     end
 
     it 'is able to detect licenses' do
-      expect(result).to match_schema
-      expect(result[:licenses]).not_to be_empty
+      expect(subject).to match_schema
+      expect(subject[:licenses]).not_to be_empty
 
       [
         { name: 'ant', licenses: ['Apache-2.0'] },
@@ -46,26 +45,25 @@ plugins {
         { name: 'sitemesh', licenses: ['Apache-1.1'] },
         { name: 'hibernate-jpa-2.1-api', licenses: ['BSD-3-Clause', 'EPL-1.0'] }
       ].each do |dependency|
-        expect(result.licenses_for(dependency[:name])).to match_array(dependency[:licenses])
+        expect(subject.licenses_for(dependency[:name])).to match_array(dependency[:licenses])
       end
     end
   end
 
   context 'when scanning a project that needs to connect to multiple TLS endpoints with different custom certificate chains' do
-    subject do
-      runner.scan(env: {
+    let(:env) do
+      {
         'ADDITIONAL_CA_CERT_BUNDLE' => x509_certificate.read,
         'PRIVATE_MAVEN_HOST' => 'maven.test'
-      })
+      }
     end
 
     before do
       runner.mount(dir: fixture_file('java/gradle/offline-environment'))
     end
 
-    specify { expect(subject).to match_schema }
-
     specify do
+      expect(subject).to match_schema
       expect(subject.dependency_names).to match_array([
         "antlr",
         "commons-beanutils",
@@ -98,20 +96,23 @@ plugins {
   end
 
   context "when scanning a gradle project with a custom option to generate a profiler report" do
-    subject { runner.scan(env: { 'GRADLE_CLI_OPTS' => '--profile' }) }
+    let(:env) { { 'GRADLE_CLI_OPTS' => '--profile' } }
 
     before do
       runner.mount(dir: fixture_file("java/gradle/java-11"))
     end
 
-    specify { expect(subject).to match_schema }
     specify { expect { subject }.to change { Dir.glob("#{runner.project_path}/build/reports/profile/profile-*.html").count }.from(0).to(1) }
-    specify { expect(subject.dependency_names).to match_array(['postgresql']) }
-    specify { expect(subject.licenses_for('postgresql')).to match_array(['BSD-2-Clause']) }
+
+    specify do
+      expect(subject).to match_schema
+      expect(subject.dependency_names).to match_array(['postgresql'])
+      expect(subject.licenses_for('postgresql')).to match_array(['BSD-2-Clause'])
+    end
   end
 
   context 'when using Java 8 with version 1.* of gradle' do
-    subject { runner.scan(env: { 'LM_JAVA_VERSION' => '8' }) }
+    let(:env) { { 'LM_JAVA_VERSION' => '8' } }
 
     before do
       runner.mount(dir: fixture_file("java/gradle/java-8"))
@@ -127,7 +128,12 @@ plugins {
 
   ['4.9', '5.6', '6.3'].each do |gradle_version|
     context "when using Java v11 with a kotlin project using gradle v#{gradle_version}" do
-      let(:report) { runner.scan(env: { 'LM_JAVA_VERSION' => '11', 'GRADLE_CLI_OPTS' => '-b build.gradle.kts' }) }
+      let(:env) do
+        {
+          'LM_JAVA_VERSION' => '11',
+          'GRADLE_CLI_OPTS' => '-b build.gradle.kts'
+        }
+      end
 
       before do
         runner.add_file('.tool-versions', "gradle #{gradle_version}")
@@ -135,9 +141,11 @@ plugins {
         runner.add_file('settings.gradle.kts', 'rootProject.name = "example"')
       end
 
-      specify { expect(report).to match_schema }
-      specify { expect(report.dependency_names).to match_array(['postgresql']) }
-      specify { expect(report.licenses_for('postgresql')).to match_array(['BSD-2-Clause']) }
+      specify do
+        expect(subject).to match_schema
+        expect(subject.dependency_names).to match_array(['postgresql'])
+        expect(subject.licenses_for('postgresql')).to match_array(['BSD-2-Clause'])
+      end
     end
   end
 
@@ -147,7 +155,7 @@ plugins {
   ].each do |item|
     item[:gradle].each do |gradle_version|
       context "when using Java v#{item[:java]} with a gradle v#{gradle_version} on a groovy project" do
-        let(:report) { runner.scan(env: { 'LM_JAVA_VERSION' => item[:java] }) }
+        let(:env) { { 'LM_JAVA_VERSION' => item[:java] } }
 
         before do
           runner.add_file('.tool-versions', "gradle #{gradle_version}")
@@ -155,9 +163,11 @@ plugins {
           runner.add_file('settings.gradle', 'rootProject.name = "example"')
         end
 
-        specify { expect(report).to match_schema }
-        specify { expect(report.dependency_names).to match_array(['postgresql']) }
-        specify { expect(report.licenses_for('postgresql')).to match_array(['BSD-2-Clause']) }
+        specify do
+          expect(subject).to match_schema
+          expect(subject.dependency_names).to match_array(['postgresql'])
+          expect(subject.licenses_for('postgresql')).to match_array(['BSD-2-Clause'])
+        end
       end
     end
   end