FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

RFE: add tests for Rust bindings by ManaSugi · Pull Request #411 · seccomp/libseccomp · GitHub

RFE: add tests for Rust bindings - #411

Open
ManaSugi wants to merge 1 commit into
seccomp:mainfrom
ManaSugi:add-rust-tests
Open

ManaSugi wants to merge 1 commit into
seccomp:mainfrom
ManaSugi:add-rust-tests

Conversation

Copy link
Copy Markdown
Contributor

Add the regression tests (1-60) for the libseccomp crate that is Rust language bindings for the libseccomp library.

You can run the tests as follows:

$ sed -i "/^AC_INIT/ s/0.0.0/9.9.9/" configure.ac
$ ./autogen.sh
$ ./configure --prefix=$(pwd)/src/.libs --enable-rust
$ make && make install
$ make check-build
$ cd tests && ./regression -m rust

Based on: #323

ManaSugi force-pushed the add-rust-tests branch 2 times, most recently from 11ff66d to 6a61aad Compare September 22, 2023 06:32
ManaSugi changed the title [WIP] tests: Add tests for Rust bindings tests: Add tests for Rust bindings Sep 22, 2023

Copy link
Copy Markdown
Contributor Author

@pcmoore @drakenclimber
All tests passed successfully. But we need to discuss how we control the CI tests with the separate Rust bindings repository as mentioned in #323.

pcmoore changed the title tests: Add tests for Rust bindings RFE: add tests for Rust bindings Nov 30, 2023
ManaSugi force-pushed the add-rust-tests branch 2 times, most recently from 81decac to aef20cc Compare February 19, 2024 11:12

coveralls commented Feb 19, 2024 •
edited
Loading

Copy link
Copy Markdown

coverage: 89.474%. remained the same
when pulling 86f5b73 on ManaSugi:add-rust-tests
into 47ca644 on seccomp:main.

Add the regression tests (1-60) for the libseccomp crate that is
Rust language bindings for the libseccomp library.

You can run the tests as follows:

```sh
$ sed -i "/^AC_INIT/ s/0.0.0/9.9.9/" configure.ac
$ ./autogen.sh
$ ./configure --prefix=$(pwd)/src/.libs --enable-rust
$ make && make install
$ make check-build
$ cd tests && ./regression -m rust
```

Based on: seccomp#323

Signed-off-by: Manabu Sugimoto <Manabu.Sugimoto@sony.com>
Signed-off-by: mayank <mayank.mrinal@sony.com>

Copy link
Copy Markdown
Contributor Author

@pcmoore @drakenclimber
Updated the year of copyright and rebased.
I'd appreciate if you could review this when you have time.

pcmoore commented Feb 20, 2024

Copy link
Copy Markdown
Member

Thank you @ManaSugi for your continued efforts here. I wanted to review things with you and @drakenclimber in the main issue tracker: #323 (comment)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL