Skip to content

Initial work on PDF tests - #335

Closed
jonjbar wants to merge 5 commits into
synopse:masterfrom
jonjbar:pdf-testing
Closed

Initial work on PDF tests#335
jonjbar wants to merge 5 commits into
synopse:masterfrom
jonjbar:pdf-testing

Conversation

@jonjbar

@jonjbar jonjbar commented Jan 24, 2025

Copy link
Copy Markdown
Contributor

This PR introduces the foundational setup for testing the PDF generation process. These tests can be improved to ensure that any future changes to the codebase do not unintentionally break the PDF generation functionality. This test suite could also double as a learning resource for new users.

@synopse

synopse commented Jan 24, 2025

Copy link
Copy Markdown
Owner

Thanks a lot for the interest!

Perhaps also worth adding the existing pdf tests of mORMot 1.

Also, ensure your code compiles with FPC, and has no warning. Your usage of TBytes/string is somewhat dubious.

@jonjbar

jonjbar commented Jan 24, 2025

Copy link
Copy Markdown
Contributor Author

New version should work with FPC, adds tests from mORMot1 and doesn't have any warning.
Recommendations to remove TBytes/string are welcome.

@synopse

synopse commented Jan 24, 2025

Copy link
Copy Markdown
Owner

Thanks a lot.
I will merge this and make some minor fixes about TEncoding and TBytes (which is not compatible with older Delphi).

@Cyberwolf73 Cyberwolf73 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Greeting

Comment thread test/mormot2tests.dpr

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Greeting

@synopse synopse added good first issue Good for newcomers pdf mormot.ui.pdf related issue tests Enhance regression tests labels Jul 6, 2026
@RODARIEDIDIER

Copy link
Copy Markdown

bonjour
j'essai de faire un pdf avec une fiche avec une imagecontrol et des tpanel ainsi que des tedit en fmx sauf quelque soit le code que je mets pourriez vous me dire si c'est possible je suis novice sur delphi je sais qu'en vlc c'est possible j'ai delphi 12 community

@synopse

synopse commented Aug 12, 2026

Copy link
Copy Markdown
Owner

bonjour j'essai de faire un pdf

Better ask your question in English on the forum, please.

TL&WR: Pas de support FMX pour l'instant, mormot.ui.pdf est du pur VCL + Windows.

landrix added a commit to landrix/mORMot2 that referenced this pull request Sep 3, 2026
Based on the initial PDF test work from pull request synopse#335.

Co-authored-by: jonjbar <17639746+jonjbar@users.noreply.github.com>
@synopse

synopse commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Has been eventually implemented from #571 pull request.
Closing.

Thanks for your initial work and interest!

@synopse synopse closed this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers pdf mormot.ui.pdf related issue tests Enhance regression tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants