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.

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

docs: add test cases for SWIOT and migrate from wiki for M2K #1873

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

AlexandraTrifan
Copy link
Contributor

@AlexandraTrifan AlexandraTrifan commented Jan 10, 2025

No description provided.

@AlexandraTrifan AlexandraTrifan force-pushed the main-doc-m2k-tests branch 3 times, most recently from c5645b2 to 8695592 Compare January 17, 2025 09:27
@AlexandraTrifan
Copy link
Contributor Author

@Andrei-Fabian-Pop ready for review

docs/tests/plugins/swiot1l/swiot1l_tests.rst Outdated Show resolved Hide resolved
docs/tests/plugins/swiot1l/swiot1l_tests.rst Outdated Show resolved Hide resolved
docs/tests/plugins/swiot1l/swiot1l_tests.rst Outdated Show resolved Hide resolved
- Click **Apply** to enter Runtime mode.

Depends on:
- Test TST.EMU.CONNECT
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above

Comment on lines 113 to 110
**Description**: This test verifies that the SWIOT plugin isSwitch modes able
to switch between Config and Runtime modes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe: "This test verifies that the SWIOT plugin is able
to switch between Config and Runtime modes." ??


**Steps:**
1. Open the AD74413R instrument.
2. Enable all the channels and ensure each channels'
Copy link
Contributor

Choose a reason for hiding this comment

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

channel's ?

- Connect loopback cables between DIO 0-7 and DIO 8-15 using the pinout diagram.

Depends on:
- Test TST.EMU.CONNECT
Copy link
Contributor

Choose a reason for hiding this comment

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

Does it depend on TST.EMU.CONNECT?

- Sample delay of -142 samples.
4. Open the Logic Analyzer trigger menu and turn on the External trigger.
Select the source as **Oscilloscope**.
5. Open the `Signal Generator<signal_generator>` instrument and generate a sinewave
Copy link
Contributor

@andreidanila1 andreidanila1 Jan 21, 2025

Choose a reason for hiding this comment

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

Maybe you wanted to put a link? In several places it is written like this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, forgot the :ref: :)

- Connect the ADALM2000 device in Scopy using the USB/network backend.

Depends on:
- Test TST.EMU.CONNECT
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above.

- Connect the ADALM2000 device in Scopy using the USB/network backend.

Depends on:
- Test TST.EMU.CONNECT
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above.

Comment on lines 67 to 68
- **Expected Result**:
- The voltage displayed on the voltmeter is between -0.050V and 0.4V.
Copy link
Contributor

@andreidanila1 andreidanila1 Jan 21, 2025

Choose a reason for hiding this comment

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

If there is only one expected result, isn't it better to put it inline?
The same for all tests.

Comment on lines 447 to 451
- Channel1 Vertical: Volts/Div: 1V/div, Position: 0V
3. TBD

**Result**: PASS/FAIL
Copy link
Contributor

Choose a reason for hiding this comment

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

??

@AlexandraTrifan AlexandraTrifan force-pushed the main-doc-m2k-tests branch 2 times, most recently from 9d27c17 to fd0c045 Compare January 22, 2025 08:54
@AlexandraTrifan AlexandraTrifan merged commit 798a051 into main-doc-test-case-struct Jan 22, 2025
23 checks passed
@AlexandraTrifan AlexandraTrifan deleted the main-doc-m2k-tests branch January 22, 2025 09:35
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