change popup text

components/frontend/src/metric/Metric.js

@@ -22,8 +22,8 @@ function MeasurementValue({ metric }) {
     if (metric.latest_measurement) {
         return (
             <Popup trigger={valueText} flowing hoverable>
-                <TimeAgoWithDate date={metric.latest_measurement.end}>Last measured</TimeAgoWithDate><br />
-                <TimeAgoWithDate date={metric.latest_measurement.start}>First measured</TimeAgoWithDate>
+                <TimeAgoWithDate date={metric.latest_measurement.end}>{metric.status ? "Metric was last measured":"Last measurement attempt"}</TimeAgoWithDate><br />
+                <TimeAgoWithDate date={metric.latest_measurement.start}>{metric.status ? "Value was first measured":"Value unknown since"}</TimeAgoWithDate>
             </Popup>
         )
     }

components/frontend/src/metric/Metric.test.js

@@ -1,6 +1,6 @@
 import React from 'react';
 import { Table } from 'semantic-ui-react';
-import { render, screen } from '@testing-library/react';
+import { fireEvent, render, screen, waitFor } from '@testing-library/react';
 import { Metric } from './Metric';
 import { DataModel } from '../context/DataModel';
 
@@ -84,3 +84,22 @@ it('renders the minutes as percentage', () => {
     expect(screen.getAllByText(/50% minutes/).length).toBe(1);
     expect(screen.getAllByText(/≦ 0% minutes/).length).toBe(1);
 });
+
+it('renders correct popups with status', async () => {
+    render_metric("violations");
+    fireEvent.mouseOver(screen.getByText(/50 violations/));
+
+    await waitFor(() => screen.getByText(/Metric was last measured/))
+    expect(screen.getAllByText(/Metric was last measured/).length).toBe(1);
+    expect(screen.getAllByText(/Value was first measured/).length).toBe(1);
+});
+
+it('renders correct popups without status', async () => {
+    report.subjects.subject_uuid.metrics.violations.status = null;
+    render_metric("violations");
+    fireEvent.mouseOver(screen.getByText(/50 violations/));
+
+    await waitFor(() => screen.getByText(/Last measurement attempt/))
+    expect(screen.getAllByText(/Last measurement attempt/).length).toBe(1);
+    expect(screen.getAllByText(/Value unknown since/).length).toBe(1);
+});

docs/src/changelog.md

@@ -21,6 +21,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 ### Changed
 
 - Use users' full name instead of their username in the change log so it's easier to see who changed what. Closes [#2930](https://github.com/ICTU/quality-time/issues/2930).
+- Improved tooltips for the measurement column in the metrics details table. Closes [#3171](https://github.com/ICTU/quality-time/issues/3171).
 
 ### Added