From 6c6908007cfb08cd31f1e1cbc94b686a2206798c Mon Sep 17 00:00:00 2001
From: Simonas <20096648+simjak@users.noreply.github.com>
Date: Tue, 2 Jan 2024 11:48:07 +0200
Subject: [PATCH] feat: precommit local hooks test
---
.pre-commit-config.yaml | 27 -----
coverage.xml | 248 ++++++++++++++++++++--------------------
scripts/run_tests.sh | 3 +
3 files changed, 127 insertions(+), 151 deletions(-)
create mode 100755 scripts/run_tests.sh
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index c8331ae3..530abf4e 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -61,30 +61,3 @@ repos:
stages:
- post-commit
- push
-
- - repo: local
- hooks:
- # - id: format
- # name: format
- # language: system
- # pass_filenames: false
- # always_run: true
- # entry: make format
- # types: [ python ]
-
- # - id: linter
- # name: linter
- # language: system
- # pass_filenames: false
- # # stages: [ push ]
- # always_run: true
- # entry: make lint
- # types: [ python ]
-
- - id: test
- name: test
- language: system
- pass_filenames: false
- # stages: [ pre-commit, push ]
- entry: make test
- types: [ python ]
diff --git a/coverage.xml b/coverage.xml
index 0f71df89..f4b8af22 100644
--- a/coverage.xml
+++ b/coverage.xml
@@ -1,12 +1,12 @@
<?xml version="1.0" ?>
-<coverage version="7.3.3" timestamp="1704187943126" lines-valid="608" lines-covered="526" line-rate="0.8651" branches-covered="0" branches-valid="0" branch-rate="0" complexity="0">
+<coverage version="7.3.3" timestamp="1704188881490" lines-valid="608" lines-covered="411" line-rate="0.676" branches-covered="0" branches-valid="0" branch-rate="0" complexity="0">
<!-- Generated by coverage.py: https://coverage.readthedocs.io/en/7.3.3 -->
<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd -->
<sources>
<source>/Users/jakit/customers/aurelio/semantic-router/semantic_router</source>
</sources>
<packages>
- <package name="." line-rate="0.9419" branch-rate="0" complexity="0">
+ <package name="." line-rate="0.7045" branch-rate="0" complexity="0">
<classes>
<class name="__init__.py" filename="__init__.py" complexity="0" line-rate="1" branch-rate="0">
<methods/>
@@ -17,7 +17,7 @@
<line number="5" hits="1"/>
</lines>
</class>
- <class name="hybrid_layer.py" filename="hybrid_layer.py" complexity="0" line-rate="1" branch-rate="0">
+ <class name="hybrid_layer.py" filename="hybrid_layer.py" complexity="0" line-rate="0.2143" branch-rate="0">
<methods/>
<lines>
<line number="1" hits="1"/>
@@ -33,94 +33,94 @@
<line number="20" hits="1"/>
<line number="23" hits="1"/>
<line number="24" hits="1"/>
- <line number="25" hits="1"/>
- <line number="27" hits="1"/>
- <line number="28" hits="1"/>
- <line number="29" hits="1"/>
- <line number="30" hits="1"/>
- <line number="32" hits="1"/>
- <line number="34" hits="1"/>
- <line number="38" hits="1"/>
+ <line number="25" hits="0"/>
+ <line number="27" hits="0"/>
+ <line number="28" hits="0"/>
+ <line number="29" hits="0"/>
+ <line number="30" hits="0"/>
+ <line number="32" hits="0"/>
+ <line number="34" hits="0"/>
+ <line number="38" hits="0"/>
<line number="40" hits="1"/>
- <line number="41" hits="1"/>
- <line number="42" hits="1"/>
- <line number="43" hits="1"/>
- <line number="44" hits="1"/>
- <line number="45" hits="1"/>
- <line number="47" hits="1"/>
+ <line number="41" hits="0"/>
+ <line number="42" hits="0"/>
+ <line number="43" hits="0"/>
+ <line number="44" hits="0"/>
+ <line number="45" hits="0"/>
+ <line number="47" hits="0"/>
<line number="49" hits="1"/>
- <line number="50" hits="1"/>
+ <line number="50" hits="0"/>
<line number="52" hits="1"/>
- <line number="54" hits="1"/>
- <line number="55" hits="1"/>
- <line number="60" hits="1"/>
- <line number="61" hits="1"/>
- <line number="62" hits="1"/>
- <line number="64" hits="1"/>
- <line number="65" hits="1"/>
- <line number="66" hits="1"/>
- <line number="70" hits="1"/>
- <line number="71" hits="1"/>
- <line number="73" hits="1"/>
- <line number="75" hits="1"/>
- <line number="76" hits="1"/>
- <line number="78" hits="1"/>
+ <line number="54" hits="0"/>
+ <line number="55" hits="0"/>
+ <line number="60" hits="0"/>
+ <line number="61" hits="0"/>
+ <line number="62" hits="0"/>
+ <line number="64" hits="0"/>
+ <line number="65" hits="0"/>
+ <line number="66" hits="0"/>
+ <line number="70" hits="0"/>
+ <line number="71" hits="0"/>
+ <line number="73" hits="0"/>
+ <line number="75" hits="0"/>
+ <line number="76" hits="0"/>
+ <line number="78" hits="0"/>
<line number="80" hits="1"/>
- <line number="82" hits="1"/>
- <line number="83" hits="1"/>
- <line number="86" hits="1"/>
- <line number="87" hits="1"/>
- <line number="90" hits="1"/>
- <line number="91" hits="1"/>
- <line number="92" hits="1"/>
- <line number="99" hits="1"/>
- <line number="106" hits="1"/>
+ <line number="82" hits="0"/>
+ <line number="83" hits="0"/>
+ <line number="86" hits="0"/>
+ <line number="87" hits="0"/>
+ <line number="90" hits="0"/>
+ <line number="91" hits="0"/>
+ <line number="92" hits="0"/>
+ <line number="99" hits="0"/>
+ <line number="106" hits="0"/>
<line number="112" hits="1"/>
- <line number="117" hits="1"/>
- <line number="118" hits="1"/>
- <line number="120" hits="1"/>
- <line number="121" hits="1"/>
- <line number="123" hits="1"/>
- <line number="125" hits="1"/>
- <line number="127" hits="1"/>
- <line number="128" hits="1"/>
- <line number="129" hits="1"/>
- <line number="131" hits="1"/>
- <line number="132" hits="1"/>
- <line number="133" hits="1"/>
- <line number="134" hits="1"/>
- <line number="136" hits="1"/>
- <line number="137" hits="1"/>
- <line number="138" hits="1"/>
- <line number="140" hits="1"/>
- <line number="141" hits="1"/>
- <line number="143" hits="1"/>
- <line number="144" hits="1"/>
+ <line number="117" hits="0"/>
+ <line number="118" hits="0"/>
+ <line number="120" hits="0"/>
+ <line number="121" hits="0"/>
+ <line number="123" hits="0"/>
+ <line number="125" hits="0"/>
+ <line number="127" hits="0"/>
+ <line number="128" hits="0"/>
+ <line number="129" hits="0"/>
+ <line number="131" hits="0"/>
+ <line number="132" hits="0"/>
+ <line number="133" hits="0"/>
+ <line number="134" hits="0"/>
+ <line number="136" hits="0"/>
+ <line number="137" hits="0"/>
+ <line number="138" hits="0"/>
+ <line number="140" hits="0"/>
+ <line number="141" hits="0"/>
+ <line number="143" hits="0"/>
+ <line number="144" hits="0"/>
<line number="146" hits="1"/>
- <line number="148" hits="1"/>
- <line number="149" hits="1"/>
- <line number="150" hits="1"/>
+ <line number="148" hits="0"/>
+ <line number="149" hits="0"/>
+ <line number="150" hits="0"/>
<line number="152" hits="1"/>
- <line number="153" hits="1"/>
- <line number="154" hits="1"/>
- <line number="155" hits="1"/>
- <line number="156" hits="1"/>
- <line number="157" hits="1"/>
- <line number="158" hits="1"/>
- <line number="160" hits="1"/>
- <line number="163" hits="1"/>
- <line number="164" hits="1"/>
- <line number="167" hits="1"/>
- <line number="168" hits="1"/>
- <line number="170" hits="1"/>
- <line number="171" hits="1"/>
+ <line number="153" hits="0"/>
+ <line number="154" hits="0"/>
+ <line number="155" hits="0"/>
+ <line number="156" hits="0"/>
+ <line number="157" hits="0"/>
+ <line number="158" hits="0"/>
+ <line number="160" hits="0"/>
+ <line number="163" hits="0"/>
+ <line number="164" hits="0"/>
+ <line number="167" hits="0"/>
+ <line number="168" hits="0"/>
+ <line number="170" hits="0"/>
+ <line number="171" hits="0"/>
<line number="173" hits="1"/>
- <line number="174" hits="1"/>
- <line number="175" hits="1"/>
- <line number="177" hits="1"/>
+ <line number="174" hits="0"/>
+ <line number="175" hits="0"/>
+ <line number="177" hits="0"/>
</lines>
</class>
- <class name="layer.py" filename="layer.py" complexity="0" line-rate="0.9081" branch-rate="0">
+ <class name="layer.py" filename="layer.py" complexity="0" line-rate="0.8541" branch-rate="0">
<methods/>
<lines>
<line number="1" hits="1"/>
@@ -194,19 +194,19 @@
<line number="114" hits="1"/>
<line number="116" hits="1"/>
<line number="120" hits="1"/>
- <line number="121" hits="1"/>
- <line number="122" hits="1"/>
+ <line number="121" hits="0"/>
+ <line number="122" hits="0"/>
<line number="124" hits="1"/>
- <line number="125" hits="1"/>
- <line number="126" hits="1"/>
- <line number="127" hits="1"/>
- <line number="128" hits="1"/>
- <line number="129" hits="1"/>
+ <line number="125" hits="0"/>
+ <line number="126" hits="0"/>
+ <line number="127" hits="0"/>
+ <line number="128" hits="0"/>
+ <line number="129" hits="0"/>
<line number="131" hits="1"/>
- <line number="132" hits="1"/>
+ <line number="132" hits="0"/>
<line number="133" hits="0"/>
- <line number="135" hits="1"/>
- <line number="136" hits="1"/>
+ <line number="135" hits="0"/>
+ <line number="136" hits="0"/>
<line number="139" hits="1"/>
<line number="140" hits="1"/>
<line number="141" hits="1"/>
@@ -328,7 +328,7 @@
<line number="30" hits="1"/>
</lines>
</class>
- <class name="route.py" filename="route.py" complexity="0" line-rate="0.9265" branch-rate="0">
+ <class name="route.py" filename="route.py" complexity="0" line-rate="0.8529" branch-rate="0">
<methods/>
<lines>
<line number="1" hits="1"/>
@@ -347,17 +347,17 @@
<line number="19" hits="1"/>
<line number="20" hits="1"/>
<line number="21" hits="1"/>
- <line number="22" hits="1"/>
- <line number="25" hits="1"/>
+ <line number="22" hits="0"/>
+ <line number="25" hits="0"/>
<line number="26" hits="1"/>
<line number="28" hits="1"/>
<line number="29" hits="1"/>
<line number="30" hits="1"/>
<line number="33" hits="1"/>
<line number="35" hits="1"/>
- <line number="36" hits="1"/>
- <line number="37" hits="1"/>
- <line number="38" hits="1"/>
+ <line number="36" hits="0"/>
+ <line number="37" hits="0"/>
+ <line number="38" hits="0"/>
<line number="41" hits="1"/>
<line number="42" hits="1"/>
<line number="43" hits="1"/>
@@ -401,7 +401,7 @@
<line number="126" hits="0"/>
</lines>
</class>
- <class name="schema.py" filename="schema.py" complexity="0" line-rate="0.9643" branch-rate="0">
+ <class name="schema.py" filename="schema.py" complexity="0" line-rate="0.8929" branch-rate="0">
<methods/>
<lines>
<line number="1" hits="1"/>
@@ -427,8 +427,8 @@
<line number="34" hits="1"/>
<line number="35" hits="1"/>
<line number="36" hits="1"/>
- <line number="37" hits="1"/>
- <line number="38" hits="1"/>
+ <line number="37" hits="0"/>
+ <line number="38" hits="0"/>
<line number="40" hits="0"/>
<line number="42" hits="1"/>
<line number="43" hits="1"/>
@@ -436,7 +436,7 @@
</class>
</classes>
</package>
- <package name="encoders" line-rate="0.9915" branch-rate="0" complexity="0">
+ <package name="encoders" line-rate="0.812" branch-rate="0" complexity="0">
<classes>
<class name="__init__.py" filename="encoders/__init__.py" complexity="0" line-rate="1" branch-rate="0">
<methods/>
@@ -461,7 +461,7 @@
<line number="12" hits="1"/>
</lines>
</class>
- <class name="bm25.py" filename="encoders/bm25.py" complexity="0" line-rate="1" branch-rate="0">
+ <class name="bm25.py" filename="encoders/bm25.py" complexity="0" line-rate="0.4865" branch-rate="0">
<methods/>
<lines>
<line number="1" hits="1"/>
@@ -481,29 +481,29 @@
<line number="21" hits="1"/>
<line number="23" hits="1"/>
<line number="25" hits="1"/>
- <line number="26" hits="1"/>
- <line number="27" hits="1"/>
- <line number="28" hits="1"/>
- <line number="29" hits="1"/>
- <line number="30" hits="1"/>
- <line number="31" hits="1"/>
- <line number="33" hits="1"/>
- <line number="35" hits="1"/>
- <line number="36" hits="1"/>
- <line number="37" hits="1"/>
- <line number="38" hits="1"/>
- <line number="39" hits="1"/>
- <line number="40" hits="1"/>
- <line number="41" hits="1"/>
- <line number="42" hits="1"/>
- <line number="43" hits="1"/>
+ <line number="26" hits="0"/>
+ <line number="27" hits="0"/>
+ <line number="28" hits="0"/>
+ <line number="29" hits="0"/>
+ <line number="30" hits="0"/>
+ <line number="31" hits="0"/>
+ <line number="33" hits="0"/>
+ <line number="35" hits="0"/>
+ <line number="36" hits="0"/>
+ <line number="37" hits="0"/>
+ <line number="38" hits="0"/>
+ <line number="39" hits="0"/>
+ <line number="40" hits="0"/>
+ <line number="41" hits="0"/>
+ <line number="42" hits="0"/>
+ <line number="43" hits="0"/>
<line number="45" hits="1"/>
- <line number="46" hits="1"/>
- <line number="47" hits="1"/>
- <line number="48" hits="1"/>
+ <line number="46" hits="0"/>
+ <line number="47" hits="0"/>
+ <line number="48" hits="0"/>
</lines>
</class>
- <class name="cohere.py" filename="encoders/cohere.py" complexity="0" line-rate="1" branch-rate="0">
+ <class name="cohere.py" filename="encoders/cohere.py" complexity="0" line-rate="0.92" branch-rate="0">
<methods/>
<lines>
<line number="1" hits="1"/>
@@ -521,8 +521,8 @@
<line number="22" hits="1"/>
<line number="23" hits="1"/>
<line number="24" hits="1"/>
- <line number="25" hits="1"/>
- <line number="26" hits="1"/>
+ <line number="25" hits="0"/>
+ <line number="26" hits="0"/>
<line number="28" hits="1"/>
<line number="29" hits="1"/>
<line number="30" hits="1"/>
diff --git a/scripts/run_tests.sh b/scripts/run_tests.sh
new file mode 100755
index 00000000..c8609f29
--- /dev/null
+++ b/scripts/run_tests.sh
@@ -0,0 +1,3 @@
+#!/bin/bash
+poetry install
+make test
--
GitLab