Skip to content

Added podman to workflows running tests.#328

Merged
kortemik merged 1 commit intoteragrep:mainfrom
Tiihott:fix_workflows
Feb 13, 2026
Merged

Added podman to workflows running tests.#328
kortemik merged 1 commit intoteragrep:mainfrom
Tiihott:fix_workflows

Conversation

@Tiihott
Copy link
Copy Markdown
Contributor

@Tiihott Tiihott commented Feb 13, 2026

Description

Aims to fix a bug causing workflows to fail because they are unable to find a valid Docker environment. Podman is enabled for workflows by enabling podman socket service, other configurations are already included in the maven pom.xml.

Includes:

  • Added run: systemctl --user start podman.socket command to CI and Coverity Scan workflows.

Resolves #321

Build fails without the fixes from PR's #325 and #327

Checklists

Testing

General

  • I have checked that my test files and functions have meaningful names.
  • I have checked that each test tests only a single behavior.
  • I have done happy tests.
  • I have tested only my own code.
  • I have tested at least all public methods.

Assertions

  • I have checked that my tests use assertions and not runtime overhead.
  • I have checked that my tests end in assertions.
  • I have checked that there is no comparison statements in assertions.
  • I have checked that assertions are in tests and not in helper functions.
  • I have checked that assertions for iterables are outside of for loops and both sides of the iteration blocks.
  • I have checked that assertions are not tested inside consumers.

Testing Data

  • I have tested algorithms and anything else with the possibility of unbound growth.
  • I have checked that all testing data is local and fully replaceable or reproducible or both.
  • I have checked that all test files are standalone.
  • I have checked that all test-specific fake objects and classes are in the test directory.
  • I have checked that my tests do not contain anything related to customers, infrastructure or users.
  • I have checked that my tests do not contain non-generic information.
  • I have checked that my tests do not do external requests and are not privately or publicly routable.

Statements

  • I have checked that my tests do not use throws for exceptions.
  • I have checked that my tests do not use try-catch statements.
  • I have checked that my tests do not use if-else statements.

Java

  • I have checked that my tests for Java uses JUnit library.
  • I have checked that my tests for Java uses JUnit utilities for parameters.

Other

  • I have only tested public behavior and not private implementation details.
  • I have checked that my tests are not (partially) commented out.
  • I have checked that hand-crafted variables in assertions are used accordingly.
  • I have tested Object Equality.
  • I have checked that I do not have any manual tests or I have a valid reason for them and I have explained it in the PR description.

Code Quality

  • I have checked that my code follows metrics set in Procedure: Class Metrics.
  • I have checked that my code follows metrics set in Procedure: Method Metrics.
  • I have checked that my code follows metrics set in Procedure: Object Quality.
  • I have checked that my code does not have any NULL values.
  • I have checked my code does not contain FIXME or TODO comments.

@Tiihott Tiihott self-assigned this Feb 13, 2026
Copy link
Copy Markdown
Contributor

@elliVM elliVM left a comment

Choose a reason for hiding this comment

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

Seems to work now

@Tiihott Tiihott requested a review from kortemik February 13, 2026 13:36
@Tiihott Tiihott added the review Issues or pull requests waiting for a review label Feb 13, 2026
@kortemik kortemik merged commit 5d72459 into teragrep:main Feb 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review Issues or pull requests waiting for a review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub workflows fail on tests because of missing valid Docker environment

3 participants