Skip to content

updatecheck demo - DO NOT COMMIT #123

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

Closed
wants to merge 4 commits into from

Conversation

decibel
Copy link
Collaborator

@decibel decibel commented Oct 23, 2016

This is a demonstration of a working updatecheck target.

That target works by pulling down UPGRADE_FROM version of pgtap via pgxn (currently defaults to 0.94.0), installing the current version, and then doing

CREATE EXTENSION pgtap VERSION :'UPGRADE_FROM';
ALTER EXTENSION pgtap UPDATE :'CURRENT_VERSION';

then running the unit tests.

The other big change in here is that all installcheck activity is now managed with pg_regress execution schedules. There are two big wins from that:

  • The extension is now loaded only once, not for every test
  • Tests are now run in parallel by default (specify PARALLEL_CONN=1 to force serial)

To support that, there's now a test/build directory where temporary build objects are put.

If we're happy with this code then I'll get it into master and start hammering out the remaining upgrade issues.

.*.swp
pgtap.sql
pgtap-core.sql
pgtap-schema.sql
uninstall_pgtap.sql
test/setup.sql
Copy link
Owner

Choose a reason for hiding this comment

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

Why is this deleted; it's a generated file. Unless you mean to put it into test/.gitignore?

@decibel
Copy link
Collaborator Author

decibel commented Oct 25, 2016 via email

@theory
Copy link
Owner

theory commented Oct 25, 2016

Oh, I guess I switched to a static file in b8c7cb2 but neglected to remove it from .gitignore. So it's not generated anymore!

@nasbyj
Copy link
Contributor

nasbyj commented Nov 19, 2019

Testing of extension update was merged in #128.

@nasbyj nasbyj closed this Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants