Skip to content

add some tests for SuffixTree #249

add some tests for SuffixTree

add some tests for SuffixTree #249

Triggered via push September 21, 2024 04:12
Status Failure
Total duration 32s
Artifacts

ci.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

5 errors and 9 warnings
no method named `get_locations` found for struct `quranize::Quranize` in the current scope: web-app/src/lib.rs#L69
error[E0599]: no method named `get_locations` found for struct `quranize::Quranize` in the current scope --> web-app/src/lib.rs:69:25 | 69 | { self.quranize.get_locations(quran).iter() } | ^^^^^^^^^^^^^ method not found in `Quranize`
mismatched types: web-app/src/lib.rs#L55
error[E0308]: mismatched types --> web-app/src/lib.rs:55:19 | 55 | .map(|(quran, explanations, location_count)| JsEncodeResult { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | expected a tuple with 2 elements, found one with 3 elements | expected due to this | = note: expected tuple `(std::string::String, std::vec::Vec<&str>)` found tuple `(_, _, _)`
this function takes 0 arguments but 1 argument was supplied: web-app/src/lib.rs#L40
error[E0061]: this function takes 0 arguments but 1 argument was supplied --> web-app/src/lib.rs:40:22 | 40 | _ => Quranize::new(min_harfs), | ^^^^^^^^^^^^^ --------- unexpected argument of type `u16` | note: associated function defined here --> /home/runner/work/quranize-rs/quranize-rs/quranize/src/lib.rs:39:12 | 39 | pub fn new() -> Self { | ^^^ help: remove the extra argument | 40 - _ => Quranize::new(min_harfs), 40 + _ => Quranize::new(), |
unresolved import `quranize::AyaGetter`: web-app/src/lib.rs#L1
error[E0432]: unresolved import `quranize::AyaGetter` --> web-app/src/lib.rs:1:16 | 1 | use quranize::{AyaGetter, Quranize}; | ^^^^^^^^^ no `AyaGetter` in the root
rust-ci
Clippy had exited with the 101 exit code
methods `find` and `collect_data` are never used: quranize/src/suffix_tree.rs#L55
warning: methods `find` and `collect_data` are never used --> quranize/src/suffix_tree.rs:55:19 | 13 | impl<'a> SuffixTree<'a> { | ----------------------- methods in this implementation ... 55 | pub(super) fn find(&self, v: usize, s: &str) -> Vec<usize> { | ^^^^ ... 65 | fn collect_data(&self, v: usize) -> Vec<usize> { | ^^^^^^^^^^^^
field `tree` is never read: quranize/src/lib.rs#L27
warning: field `tree` is never read --> quranize/src/lib.rs:27:5 | 26 | pub struct Quranize { | -------- field in this struct 27 | tree: suffix_tree::SuffixTree<'static>, | ^^^^ | = note: `#[warn(dead_code)]` on by default
unused variable: `it`: quranize/src/suffix_tree/suffix_iter.rs#L2
warning: unused variable: `it` --> quranize/src/suffix_tree/suffix_iter.rs:2:9 | 2 | let it = s.char_indices(); | ^^ help: if this is intentional, prefix it with an underscore: `_it` | = note: `#[warn(unused_variables)]` on by default
rust-ci
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
rust-ci
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
rust-ci
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
rust-ci
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
rust-ci
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
rust-ci
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/