Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add new expect action #501

Closed
wants to merge 11 commits into from
Closed

Add new expect action #501

wants to merge 11 commits into from

Commits on Jun 25, 2024

  1. WIP: expect step

    @actions-user @l50
    actions-user authored and l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    752266e View commit details
    Browse the repository at this point in the history

  2. Working with an awkward refactor

    @actions-user @l50
    actions-user authored and l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    a1070a0 View commit details
    Browse the repository at this point in the history

  3. Refactored expect.yaml and step.go to remove type field from step s… 

    …tructs

**Changed:**

- Refactored `expect.yaml` to remove `type` fields from steps.
- Updated `basicstep.go` and `expectstep.go` to remove `type` field from step
  structs and initialization.
- Modified `step.go` to dynamically determine step types without `type` field.
- Enhanced `expectstep_test.go` with additional test cases for ExpectStep.

**Added:**

- Introduced environment and output fields to ExpectStep struct.
- Added error handling for missing responses in ExpectStep validation.

**Removed:**

- Removed redundant log messages from `step.go`.
    @actions-user @l50
    actions-user authored and l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    1e2acfe View commit details
    Browse the repository at this point in the history

  4. Fully working ExpectStep

    @actions-user @l50
    actions-user authored and l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    bbb284a View commit details
    Browse the repository at this point in the history

  5. Updated expectstep.go and step.go with enhanced documentation and… 

    … attributes.

**Added:**

- Detailed comments for ExpectStep attributes in `expectstep.go`
- Detailed comments for Response attributes in `expectstep.go`
- Detailed comments for all ExpectStep methods in `expectstep.go`
- Check for ambiguous types in ParseAction method of `step.go`
- Additional fields (File, TTP, EditFile) in ParseAction method of `step.go`

**Changed:**

- Renamed error messages in `expectstep.go` to lowercase for consistency
- Updated copyright year in `step.go`
- Refactored comments to adhere to GoDoc standards in `expectstep.go`
    @actions-user @l50
    actions-user authored and l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    832329f View commit details
    Browse the repository at this point in the history

  6. Added directory change handling and refactored test script creation 

    **Added:**

- Directory change handling in ExpectStep `Execute` method with defer function
- New test case for ExpectStep with directory change verification
- Cleanup step execution in test cases to verify script removal

**Changed:**

- Refactored `createTestScript` to accept script content as a parameter
- Updated test cases to use dynamic script content
- Adjusted ExpectStep validation to remove redundant comments

**Removed:**

- Removed license header from `expectstep_test.go` for simplicity
    @l50
    l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    27fabe2 View commit details
    Browse the repository at this point in the history

  7. Update to use non-forked go mod - reducing dependencies

    @actions-user @l50
    actions-user authored and l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    377a4f9 View commit details
    Browse the repository at this point in the history

  8. Final round of squashing go mods

    @l50
    l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    9993697 View commit details
    Browse the repository at this point in the history

  9. Remove accidentally tripled pre-commit hook

    @l50
    l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    df5eae6 View commit details
    Browse the repository at this point in the history

  10. Update test action to do coverage testing + bug fixes

    @l50
    l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    6101afe View commit details
    Browse the repository at this point in the history

  11. Remove licenses action in favor of testing in pre-commit

    @l50
    l50 committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    55fbabb View commit details
    Browse the repository at this point in the history