Skip to content

Tests for primary constuctors,passing cases#2009

Merged
RobertvanderHulst merged 1 commit into
X-Sharp:devfrom
dimitrakokkali:primary_tests
Jul 17, 2026
Merged

Tests for primary constuctors,passing cases#2009
RobertvanderHulst merged 1 commit into
X-Sharp:devfrom
dimitrakokkali:primary_tests

Conversation

@dimitrakokkali

Copy link
Copy Markdown

Add tests for primary constructors: (for VO/Core and for net framework/sdk)

  • class without primary constructor and clipper constructor //VO
  • class with just a primary constructor
  • class with primary constructor and two typed constructors
  • class with just a typed constructor
  • same for structures
  • same for record class and record structure //net sdk

Notes:

  • record primary constructors only supported with .net SDK
  • clipper constructor not supported in records - throws internal error !!!
  • VO assembly references have paths local to my PC - they do not match existing paths !!!
  • I have added the tests under "NEW"
  • Tests were not included in "Runtime Tests.prg"; I think we don't need a separate application for .NET 8 tests

Must-fail tests will be added in a follow-up.

Let me know of any comments.

@RobertvanderHulst

Copy link
Copy Markdown
Member

Looks good, although Chris won't like the fact that the first test is called D000 ;)

@RobertvanderHulst
RobertvanderHulst merged commit e8061b1 into X-Sharp:dev Jul 17, 2026
1 check passed
@cpyrgas

cpyrgas commented Jul 17, 2026

Copy link
Copy Markdown

Looks good, although Chris won't like the fact that the first test is called D000 ;)

Haha indeed :)
Well, apart from the zero based name indexing, very good job Dimitra!

Just one small thing, please use some a descriptive name for the tests ( Application-> Properties -> General page, Application Name ), so we can easily tell with a glance what the test is for

Also, Robert, I have added long ago the .xicfg extension in gitignore, but the file xSharp Tests30.xicfg (local project settings, similar to the .suo file for VS) is already in the repo. How to remove it from source control, just a simple delete of the file and push of changes will be enough? It will be created again next time the project is opened.

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.

3 participants