Skip to content

Commit

Permalink
Merge pull request #383 from opencybersecurityalliance/cleanup-gramma…
Browse files Browse the repository at this point in the history
…r-simplestring

Cleanup grammar simplestring
  • Loading branch information
subbyte authored Jul 26, 2023
2 parents fed9dbc + 0184de2 commit 9b2680f
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ install_requires =
nest-asyncio>=1.5.6
stix-shifter==5.3.1
stix-shifter-utils==5.3.1
firepit>=2.3.24
firepit>=2.3.25
tests_require =
pytest

Expand Down
2 changes: 0 additions & 2 deletions src/kestrel/syntax/kestrel.lark
Original file line number Diff line number Diff line change
Expand Up @@ -283,8 +283,6 @@ ESCAPED_STRING: "\"" _STRING_ESC_INNER "\""
| "'" _STRING_ESC_INNER "'"
ESCAPED_STRING_WS: WS* ESCAPED_STRING WS*

SIMPLE_STRING: ECNAME

// nearly Python string, but no [ubf]? as prefix options
// check Lark example of Python parser for reference
advanced_string: /(r?)("(?!"").*?(?<!\\)(\\\\)*?"|'(?!'').*?(?<!\\)(\\\\)*?')/
Expand Down
1 change: 0 additions & 1 deletion tests/test_stixshifter_translator.py
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,6 @@ def test_fast_translate():
assert translator.is_alive() == False


@pytest.mark.skip(reason="kestrel v1.7.1 released with issue #370 unfixed")
def test_stixshifter_fast_translate_with_parquet_writing_to_disk(tmpdir):
query_id = "8df266aa-2901-4a94-ace9-a4403e310fa1"
check_module_availability(CONNECTOR_NAME)
Expand Down

0 comments on commit 9b2680f

Please sign in to comment.