Skip to content

Add ALKiln automated tests#104

Open
byronmcquain wants to merge 4 commits intomainfrom
automated_testing
Open

Add ALKiln automated tests#104
byronmcquain wants to merge 4 commits intomainfrom
automated_testing

Conversation

@byronmcquain
Copy link
Copy Markdown

@byronmcquain byronmcquain commented Nov 6, 2025

Added these files:

  • tests/features/interviews_run.feature
  • .github/workflows/run_form_tests.yml

Want to disable the tests? See documentation for ALKiln tests at https://suffolklitlab.github.io/docassemble-AssemblyLine-documentation/docs/automated_integrated_testing.

Closes #103

@byronmcquain byronmcquain requested a review from plocket November 6, 2025 21:18
Copy link
Copy Markdown
Contributor

@plocket plocket left a comment

Choose a reason for hiding this comment

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

Thank you for this! Unfortunately, it seems this is a more complex picture than it first appeared. There were already e-filing .feature file tests on here despite there being no GitHub workflow. I'll check up on this and see what the next step might be.

@plocket plocket added the question Further information is requested label Nov 7, 2025
@BryceStevenWilley BryceStevenWilley force-pushed the automated_testing branch 2 times, most recently from de0c330 to 7c13406 Compare March 5, 2026 17:09
Copy link
Copy Markdown
Contributor

@BryceStevenWilley BryceStevenWilley left a comment

Choose a reason for hiding this comment

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

LGTM, trying to get the tests running again to check what we'd need to change about the e-filing ones.

@BryceStevenWilley
Copy link
Copy Markdown
Contributor

BryceStevenWilley commented Mar 5, 2026

Tried making progress on this, and running the tests locally did find some issues (see #109), but I can't get it to run fully due to some combination of issues in docassemble versions:

Will have to set this down and revisit when those issues are patched.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ALKiln tests

3 participants