meli
/
meli
8
Fork 13
Code Issues 10 Pull Requests 10 Actions Packages Projects 1 Releases 18 Wiki Activity

melib/tests: merge integration tests in one crate #336

Merged
Manos Pitsidianakis merged 3 commits from melib-merge-int-tests into master 2024-01-01 12:07:31 +02:00
Conversation 0 Commits 3 Files Changed 11
Manos Pitsidianakis commented 2024-01-01 10:48:17 +02:00
Owner

Saves about 0.5 seconds from compilation and runtime.

Signed-off-by: Manos Pitsidianakis manos@pitsidianak.is

Summary of the PR

melib/tests: merge integration tests in one crate

Requirements

Before submitting your PR, please make sure you have addressed the following requirements:

  • All commits in this PR are signed (with git commit -s), and the commit has a message describing the motivation behind the change, if appropriate.
  • All added/changed public-facing functionality, especially configuration options, are documented in the manual pages.
  • Any newly added unsafe code is properly documented.
  • Each commit has been formatted with rustfmt. Run make fmt in the project root.
  • Each commit has been linted with clippy. Run make lint in the project root.
  • Each commit does not break any test. Run make test in the project root. If you have cargo-nextest installed, you can run cargo nextest run --all --no-fail-fast --all-features --future-incompat-report instead.
Saves about 0.5 seconds from compilation and runtime. Signed-off-by: Manos Pitsidianakis <manos@pitsidianak.is> <!-- If your PR is ready to merge/review, remove the `WIP: ` prefix from the title. --> ### Summary of the PR melib/tests: merge integration tests in one crate ### Requirements Before submitting your PR, please make sure you have addressed the following requirements: * [x] All commits in this PR are signed (with `git commit -s`), and the commit has a message describing the motivation behind the change, if appropriate. * [x] All added/changed public-facing functionality, especially configuration options, are documented in the manual pages. * [x] Any newly added `unsafe` code is properly documented. * [x] Each commit has been formatted with `rustfmt`. Run `make fmt` in the project root. * [x] Each commit has been linted with `clippy`. Run `make lint` in the project root. * [x] Each commit does not break any test. Run `make test` in the project root. If you have `cargo-nextest` installed, you can run `cargo nextest run --all --no-fail-fast --all-features --future-incompat-report` instead.
Manos Pitsidianakis added 1 commit 2024-01-01 10:48:18 +02:00
73d5b24e98
melib/tests: merge integration tests in one crate 
Saves about 0.5 seconds from compilation and runtime.

Signed-off-by: Manos Pitsidianakis <manos@pitsidianak.is>
Manos Pitsidianakis added 1 commit 2024-01-01 11:04:00 +02:00
Cargo manifest lints / Lint Cargo manifests on ${{ matrix.build }} (linux-amd64, ubuntu-latest, stable, x86_64-unknown-linux-gnu) (pull_request) Successful in 15m55s Details
Run Tests / Test on ${{ matrix.build }} (linux-amd64, ubuntu-latest, stable, x86_64-unknown-linux-gnu) (pull_request) Successful in 20m39s Details
Run cargo lints / Lint on ${{ matrix.build }} (linux-amd64, ubuntu-latest, stable, x86_64-unknown-linux-gnu) (pull_request) Successful in 7m47s Details
8a95febb78
CI: set debuginfo=0 in test/lint builds 
Signed-off-by: Manos Pitsidianakis <manos@pitsidianak.is>
Manos Pitsidianakis added 1 commit 2024-01-01 11:39:18 +02:00
Run cargo lints / Lint on ${{ matrix.build }} (linux-amd64, ubuntu-latest, stable, x86_64-unknown-linux-gnu) (pull_request) Successful in 9m54s Details
Cargo manifest lints / Lint Cargo manifests on ${{ matrix.build }} (linux-amd64, ubuntu-latest, stable, x86_64-unknown-linux-gnu) (pull_request) Successful in 14m46s Details
Run Tests / Test on ${{ matrix.build }} (linux-amd64, ubuntu-latest, stable, x86_64-unknown-linux-gnu) (pull_request) Successful in 14m28s Details
33408146a1
Fix feature permutation mis-compilations found with cargo-hack 
Signed-off-by: Manos Pitsidianakis <manos@pitsidianak.is>
Manos Pitsidianakis merged commit 33408146a1 into master 2024-01-01 12:07:31 +02:00
Manos Pitsidianakis deleted branch melib-merge-int-tests 2024-01-01 12:07:31 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
IMAP

Related to the IMAP backend   
JMAP

Related to the JMAP backend   
Maildir

Related to the Maildir backend   
Retired

Issues that have been open so long we might as well drop them   
User Experience

Related to use and workflow with meli   
User Interface

Related to the terminal user interface   
bsd

*BSD target   
bug

Something is not working   
contacts
   
currently worked on

Stuff I am working on right now   
documentation
   
duplicate

This issue or pull request already exists   
easy

Non-complex issues   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
linux-gnu

target   
macos

apple-darwin target   
mbox

Related to the mbox backend   
notmuch

Related to the notmuch backend   
question

More information is needed   
security
   
wishlist

New features that would be cool to have, but are not priority   
wontfix

This won't be fixed
No Label
IMAP
JMAP
Maildir
Retired
User Experience
User Interface
bsd
bug
contacts
currently worked on
documentation
duplicate
easy
enhancement
help wanted
invalid
linux-gnu
macos
mbox
notmuch
question
security
wishlist
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: meli/meli#336
There is no content yet.
Delete Branch "melib-merge-int-tests"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?