Skip to content

Support package-private classes and constructors in AssembleModelGenerator #356

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

Merged
merged 1 commit into from
Jun 30, 2022

Conversation

EgorkaKulikov
Copy link
Collaborator

Description

Classes have package-private modifier by default, it is good enough to construct assemble models.
Java reflection does not allow to call newInstance method for a non-public class, but it may be fixed with withAccessibility modifier

Fixes # (346)

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Automated Testing

Run AssembleModelGeneratorTests from standard pipeline

Manual Scenario

Could be tested on classes with default modifier of class or it constructors.

@EgorkaKulikov EgorkaKulikov linked an issue Jun 30, 2022 that may be closed by this pull request
@EgorkaKulikov EgorkaKulikov force-pushed the egor/asmodgen_class_modifier branch from 28aba01 to d8db6ff Compare June 30, 2022 08:19
@EgorkaKulikov EgorkaKulikov enabled auto-merge (squash) June 30, 2022 08:28
@EgorkaKulikov EgorkaKulikov force-pushed the egor/asmodgen_class_modifier branch from e134901 to 463c227 Compare June 30, 2022 13:21
@EgorkaKulikov EgorkaKulikov merged commit 0fb3fc3 into main Jun 30, 2022
@EgorkaKulikov EgorkaKulikov deleted the egor/asmodgen_class_modifier branch June 30, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Use constructor call for package-private class
2 participants