Skip to content

systemtests: add a test for the jobs last status table in webui#1024

Merged
pstorz merged 2 commits intomasterfrom
dev/fbergkemper/master/test-dashboard-table-records
Dec 16, 2021
Merged

systemtests: add a test for the jobs last status table in webui#1024
pstorz merged 2 commits intomasterfrom
dev/fbergkemper/master/test-dashboard-table-records

Conversation

@frb121
Copy link
Contributor

@frb121 frb121 commented Dec 15, 2021

This is just a simple test to verify if we got an entry for a
particular client we expect to be present in the jobs last status
table on the dashboard.

Thank you for contributing to the Bareos Project!

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

General
  • PR name is meaningful
  • Purpose of the PR is understood
  • Separate commit for this PR in the CHANGELOG.md, PR number referenced is same
  • Commit descriptions are understandable and well formatted
Source code quality
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR
  • bareos-check-sources --since-merge does not report any problems
  • git status should not report modifications in the source tree after building and testing
Tests
  • Decision taken that a system- or unittest is required (if not, then remove this paragraph)
  • The decision towards a systemtest is reasonable compared to a unittest
  • Testname matches exactly what is being tested
  • Output of the test leads quickly to the origin of the fault

@frb121 frb121 self-assigned this Dec 15, 2021
@frb121 frb121 requested a review from pstorz December 15, 2021 10:00
@pstorz pstorz force-pushed the dev/fbergkemper/master/test-dashboard-table-records branch from 6a83897 to fe57c74 Compare December 16, 2021 16:09
This is just a simple test to verify if we got an entry for a
particular client we expect to be present in the jobs last status
table on the dashboard.
@frb121 frb121 force-pushed the dev/fbergkemper/master/test-dashboard-table-records branch from fe57c74 to 23a9ebd Compare December 16, 2021 16:10
@pstorz pstorz force-pushed the dev/fbergkemper/master/test-dashboard-table-records branch from 23a9ebd to 942bc6e Compare December 16, 2021 16:15
Copy link
Member

@pstorz pstorz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work!

@pstorz pstorz merged commit 51339c6 into master Dec 16, 2021
@pstorz pstorz deleted the dev/fbergkemper/master/test-dashboard-table-records branch December 16, 2021 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants