Skip to content

Commit

Permalink
task/TUP-713: Show inactive allocations on active projects and distin…
Browse files Browse the repository at this point in the history
…guish by 'status' (#450)

* Show inactive allocations on active projects and distinguish by 'status'

* formatting

* fix tests

---------

Co-authored-by: Wesley B <[email protected]>
  • Loading branch information
jarosenb and wesleyboar authored Apr 3, 2024
1 parent d758e35 commit 48e1a11
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ export const ProjectsListingAllocationTable: React.FC<{
project: ProjectsRawSystem;
}> = ({ project }) => {
const allocations = project.allocations || [];
const isActive = allocations.some((a) => a.status === 'Active');

return (
<table className={styles['allocations-table']}>
Expand All @@ -20,21 +19,20 @@ export const ProjectsListingAllocationTable: React.FC<{
<th>Systems</th>
<th>Awarded</th>
<th>Used</th>
<th>Status</th>
<th>Expires</th>
</tr>
</thead>
<tbody>
{allocations
// Show expired allocations if the project is inactive.
.filter((a) => (isActive ? a.status === 'Active' : true))
.map((allocation) => (
<tr key={allocation.id}>
<td>{allocation.resource}</td>
<td>{allocation.total} SU</td>
<td>{allocation.used} SU</td>
<td>{formatDate(allocation.end)}</td>
</tr>
))}
{allocations.map((allocation) => (
<tr key={allocation.id}>
<td>{allocation.resource}</td>
<td>{allocation.total} SU</td>
<td>{allocation.used} SU</td>
<td>{allocation.status ?? '-'}</td>
<td>{formatDate(allocation.end)}</td>
</tr>
))}
</tbody>
</table>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ describe('Project Details Component', () => {
expect(columnHeaders[0].textContent).toEqual('Systems');
expect(columnHeaders[1].textContent).toEqual('Awarded');
expect(columnHeaders[2].textContent).toEqual('Used');
expect(columnHeaders[3].textContent).toEqual('Expires');
expect(columnHeaders[3].textContent).toEqual('Status');
expect(columnHeaders[4].textContent).toEqual('Expires');
expect(getByText('Lonestar6')).toBeDefined();
expect(getByText('10 SU')).toBeDefined();
expect(getByText('0 SU')).toBeDefined();
Expand Down

0 comments on commit 48e1a11

Please sign in to comment.