Merge pull request #6654 from NHSDigital/alistair/teams-list-add-cut-off-date

Add `nr_cut_off_date` and `type` to `bin/mavis teams list`

Author: alistair-white-horne-tng

app/lib/mavis_cli/teams/list.rb

@@ -26,15 +26,24 @@ def call(ods_code: nil)
 
         rows =
           teams.find_each.map do |team|
-            team.slice(:id, :name, :workgroup).merge(
+            team.slice(:id, :name, :workgroup, :type).merge(
               ods_code: team.organisation.ods_code,
-              programmes: team.programmes.map(&:name).join(", ")
+              programmes: team.programmes.map(&:name).join(", "),
+              nr_cut_off_date: team.national_reporting_cut_off_date
             )
           end
 
         puts TableTennis.new(
                rows,
-               columns: %i[id name ods_code workgroup programmes],
+               columns: %i[
+                 id
+                 name
+                 type
+                 ods_code
+                 workgroup
+                 programmes
+                 nr_cut_off_date
+               ],
                zebra: true
              )
       end

spec/features/cli_teams_list_spec.rb

@@ -23,11 +23,18 @@ def given_a_couple_organisations_exist
   end
 
   def and_there_are_teams_in_the_organisations
-    @programme = Programme.sample
+    @programme = Programme.menacwy
     @team1 =
       create(:team, organisation: @organisation1, programmes: [@programme])
     @team2 =
       create(:team, organisation: @organisation2, programmes: [@programme])
+    @team_national_reporting =
+      create(
+        :team,
+        :national_reporting,
+        national_reporting_cut_off_date: 1.day.ago
+      )
+    @team_support = create(:team, :support)
   end
 
   def when_i_run_the_list_teams_command
@@ -46,11 +53,24 @@ def when_i_run_the_list_teams_command_with_an_ods_code
 
   def then_i_should_see_the_list_of_teams
     expect(@output).to include(@team1.name)
+    expect(@output).to include(@team1.type)
     expect(@output).to include(@organisation1.ods_code)
     expect(@output).to include(@team1.workgroup)
+
     expect(@output).to include(@team2.name)
+    expect(@output).to include(@team2.type)
     expect(@output).to include(@organisation2.ods_code)
     expect(@output).to include(@team2.workgroup)
+
+    expect(@output).to include(@team_national_reporting.name)
+    expect(@output).to include(@team_national_reporting.type)
+    expect(@output).to include(
+      @team_national_reporting.national_reporting_cut_off_date.to_s
+    )
+
+    expect(@output).to include(@team_support.name)
+    expect(@output).to include(@team_support.type)
+
     expect(@output).to include(@programme.name).twice
   end
 

spec/support/capture_output.rb

@@ -16,6 +16,11 @@ def capture_output(input: nil)
           end
         stub_const("ProgressBar::Output::DEFAULT_OUTPUT_STREAM", output)
 
+        # Pretend that the terminal window is wide, so table cells aren't truncated.
+        allow(TableTennis::Util::Console).to receive(:winsize).and_return(
+          [48, 220]
+        )
+
         yield
 
         output.string