Skip to content

[CLIENT-5049] Convert existing code examples in examples/ folder to inherit from a base class for boilerplate setup - #1149

Open
juliannguyen4 wants to merge 64 commits into
devfrom
CLIENT-5049-convert-existing-code-examples-to-use-harness-class
Open

[CLIENT-5049] Convert existing code examples in examples/ folder to inherit from a base class for boilerplate setup#1149
juliannguyen4 wants to merge 64 commits into
devfrom
CLIENT-5049-convert-existing-code-examples-to-use-harness-class

Conversation

@juliannguyen4

@juliannguyen4 juliannguyen4 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

This is preparation for the aerospike.com documentation to reuse examples from examples/client.

CI/CD: Validate all code examples in examples/client

Notes

Code examples will be tweaked later on once they are included in the aerospike.com docs, so they may not be in a final state.

Extra changes

Remove the outdated async examples

Risks

Code examples can no longer take in parameters, so users must manually tweak the examples before running them.

TODO

  • ttl.py needs addressing
  • aggregate.py

Out of scope

  • Verification of code example expected outputs (spoke with Justin)
  • Running run_all_examples fails if not run from repo root directory

@codecov-commenter

codecov-commenter commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.39%. Comparing base (57cbf26) to head (e145b77).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1149      +/-   ##
==========================================
- Coverage   85.39%   85.39%   -0.01%     
==========================================
  Files         101      101              
  Lines       14615    14614       -1     
==========================================
- Hits        12480    12479       -1     
  Misses       2135     2135              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…. TODO - Because of the C# client's fixture system, I'm wondering if we can just use pytest
…t support this. Also remove outdated python 2 compatibility code
@juliannguyen4
juliannguyen4 marked this pull request as ready for review July 29, 2026 17:07
…les while logged in. I believe customers starting out with aerospike would use CE before trying a build with EE features like security.
…ust have the sys-admin role. We can add it since admin user has user-admin role by default.
…esn't take long to collect all the example classes.
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