Skip to content

CG-10450 Integration tests improvements #145

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
Merged
merged 38 commits into from
Jan 28, 2025
Merged

CG-10450 Integration tests improvements #145

merged 38 commits into from
Jan 28, 2025

Conversation

tkfoss
Copy link
Contributor

@tkfoss tkfoss commented Jan 28, 2025

Motivation

We want to allow running integration tests for unauthenticated contributors.

Content

  • removed installing ghcli from the code
  • make sure REPO_ID_TO_URL is populated properly
  • don't prompt user for auth if not using lfs

Testing

Manual testing.

Please check the following before marking your PR as ready for review

  • I have added tests for my changes
  • I have updated the documentation or added new documentation as needed
  • I have read and agree to the Contributor License Agreement

tkucar and others added 30 commits January 24, 2025 01:26
patch
22b095b
add nicer pytest skip message
4490142
Merge remote-tracking branch 'origin/develop' into tom-cg-10450-enabl…
fd3a5a9 
…e-unauthenticated-codemod-test-runs
skip
2d10ce4
crude fix; skip when no node/npm
c109e83
Merge remote-tracking branch 'origin/develop' into tom-cg-10450-enabl…
5a2d5e8 
…e-unauthenticated-codemod-test-runs
@rushilpatel0
fix: url for graph widget (#114)
4cb549f 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)
@jemeza-codegen @jayhack
Adds missing return types, Styling modifications (CG-10500, CG-10499,…
cb99fc1 
… CG-10498) (#98)

# Motivation
Methods were missing return types, visibility of types in docs in light
mode was unsatisfactory

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [x] I have added tests for my changes
- [x] I have updated the documentation or added new documentation as
needed
- [x] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: Jay Hack <[email protected]>
Co-authored-by: codegen-bot <[email protected]>
Revert "skip tests requiring auth, remove hardcoded gh install" (#115)
a47158b 
Reverts #47
@caroljung-cg
Remove gs from codegen pkg (#118)
62f01bd 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [x] I have added tests for my changes
- [x] I have updated the documentation or added new documentation as
needed
- [x] I have read and agree to the [Contributor License
Agreement](../CLA.md)

Co-authored-by: codegen-bot <[email protected]>
@Edward-Codegen
Deprecate default_branch from LocalRepoOperator (#116)
bd0d9f2 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

Co-authored-by: codegen-bot <[email protected]>
@eacodegen
Update channel (#120)
737ce38 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

Co-authored-by: codegen-bot <[email protected]>
@caroljung-cg
Move remote git tests into tests/integration (#122)
193c4d7
@tawsifkamal
Tawsif add support for codebase exports (#117)
bca3332 
# Motivation
<!-- Why is this change necessary? -->
- adds exports to codebase.exports and directory.exports + directory.get_export(name) for typescript codebases
- Made a new PR so that I didn't have to deal with merge conflicts from
graphsitter renaming to codegen-sdk
- Fixed all comments from previous PR
- Added better typehints to ensure that exports are meant only for
typescript codebases and typescript specific directories

---------

Co-authored-by: codegen-bot <[email protected]>
@rushilpatel0 @codegen-team
update graph widget url (#123)
89f65a5 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: Codegen Team (Internal) <[email protected]>
@rushilpatel0 @codegen-team
Rpatel/update graph widget url (#126)
f25415f 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: Codegen Team (Internal) <[email protected]>
@eacodegen @bagel897
Update pyproject.toml metadata (#127)
bdb3c7c 
Co-authored-by: bagel897 <[email protected]>
@jayhack
feat: codegen init creates + perists .codegen/.venv (#124)
23d702b 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: codegen-bot <[email protected]>
@joelaguero
Update README.md (#131)
317c7ea 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)
@eacodegen @bagel897
@caroljung-cg @rushilpatel0 @christinewangcw @renovate @jayhack @jemeza-codegen
Make codebase.reset only reset changes made by the sdk (#74)
c04b0f4 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: bagel897 <[email protected]>
Co-authored-by: codegen-bot <[email protected]>
Co-authored-by: Carol Jung <[email protected]>
Co-authored-by: Rushil Patel <[email protected]>
Co-authored-by: Christine Wang <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jay Hack <[email protected]>
Co-authored-by: tomcodgen <[email protected]>
Co-authored-by: jemeza-codegen <[email protected]>
@Edward-Codegen
Automatically determine base_path and throw error on non-git repos (#121
bfc77ee 
)

# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

Co-authored-by: codegen-bot <[email protected]>
@eacodegen
Fix link (#132)
a97f64c 
Co-authored-by: codegen-bot <[email protected]>
@eacodegen @bagel897
Fix gradio (#130)
fffe207 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: bagel897 <[email protected]>
Co-authored-by: codegen-bot <[email protected]>
@joelaguero
Update README.md (#134)
878abc7 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)
@jayhack
feat: enables codegen create -d (#135)
f0dab50 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: codegen-bot <[email protected]>
@christinewangcw
chore: CG-10545 remove codebase.commit warning (#136)
7b4e8cc
@eacodegen
Add posthog and override (#140)
41da296 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

Co-authored-by: codegen-bot <[email protected]>
@caroljung-cg
Sync sandbox runner code (#141)
5832f60
@kopekC
Documentation for set_session_options (#137)
e055880 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [x] I have added tests for my changes
- [x] I have updated the documentation or added new documentation as
needed
- [x] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: KopekC <[email protected]>
@Edward-Codegen
New Codebase Init Flow (#139)
f4af64b 
# Motivation
<!-- Why is this change necessary? -->

# Content
<!-- Please include a summary of the change -->
# Testing
<!-- How was the change tested? -->
# Please check the following before marking your PR as ready for review

- [ ] I have added tests for my changes
- [ ] I have updated the documentation or added new documentation as
needed
- [ ] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: codegen-bot <[email protected]>
renovate bot and others added 2 commits January 28, 2025 03:56
@renovate
chore(deps): update dependency @types/node to v22.12.0 (#142)
8b2f0f3 
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[@types/node](https://redirect.github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node)
([source](https://redirect.github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node))
| [`22.10.10` ->
`22.12.0`](https://renovatebot.com/diffs/npm/@types%2fnode/22.10.10/22.12.0)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@types%2fnode/22.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@types%2fnode/22.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@types%2fnode/22.10.10/22.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@types%2fnode/22.10.10/22.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/codegen-sh/codegen-sdk).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xMjUuMSIsInVwZGF0ZWRJblZlciI6IjM5LjEyNS4xIiwidGFyZ2V0QnJhbmNoIjoiZGV2ZWxvcCIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
skip tests that dont use expected fixture
49bf424
@tkfoss tkfoss requested a review from codegen-team as a code owner January 28, 2025 03:01
@linear Linear
Copy link

linear bot commented Jan 28, 2025

CG-10450 Integration tests improvements

Enable unauthenticated codemod test runs

This requires auth to github.codegen.app:

uv run pytest tests/codemod/test_codemods.py -n auto

context: https://thecodegen.slack.com/archives/C070DA3BNH3/p1737663274013509?thread_ts=1737660890.493439&cid=C070DA3BNH3

Sanity check

Also: it looks like this codemod uses an outdated API? Assignment.right does not exist. Why is this test not failing

https://github.com/codegen-sh/codegen-sdk/blob/e79170cef354d21245bf028c16acc4a7df2[…]onvert_attribute_to_decorator/convert_attribute_to_decorator.py

bagel897
bagel897 suggested changes Jan 28, 2025
View reviewed changes
@tkfoss tkfoss merged commit 10f68cc into develop Jan 28, 2025
8 of 13 checks passed
@tkfoss tkfoss deleted the tom-cg-10450-enable-unauthenticated-codemod-test-runs branch January 28, 2025 20:44
jayhack added a commit that referenced this pull request Feb 9, 2025
@rushilpatel0 @jemeza-codegen
@jayhack @caroljung-cg @Edward-Codegen @eacodegen @tawsifkamal @codegen-team @bagel897 @joelaguero @christinewangcw @renovate @kopekC
CG-10450 Integration tests improvements (#145)
fe14c88 
# Motivation
We want to allow running integration tests for unauthenticated
contributors.

# Content
- removed installing ghcli from the code
- make sure REPO_ID_TO_URL is populated properly
- don't prompt user for auth if not using lfs

# Testing
Manual testing.

# Please check the following before marking your PR as ready for review

- [x] I have added tests for my changes
- [x] I have updated the documentation or added new documentation as
needed
- [x] I have read and agree to the [Contributor License
Agreement](../CLA.md)

---------

Co-authored-by: Rushil Patel <[email protected]>
Co-authored-by: jemeza-codegen <[email protected]>
Co-authored-by: Jay Hack <[email protected]>
Co-authored-by: codegen-bot <[email protected]>
Co-authored-by: Carol Jung <[email protected]>
Co-authored-by: Edward-Codegen <[email protected]>
Co-authored-by: eacodegen <[email protected]>
Co-authored-by: Tawsif Kamal <[email protected]>
Co-authored-by: Codegen Team (Internal) <[email protected]>
Co-authored-by: bagel897 <[email protected]>
Co-authored-by: Joel Aguero <[email protected]>
Co-authored-by: Christine Wang <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Edo Pujol <[email protected]>
Co-authored-by: KopekC <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bagel897 bagel897 bagel897 approved these changes

@codegen-team codegen-team Awaiting requested review from codegen-team codegen-team is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.