search

trueagi-io / metta-morph

Metta-morph (from Metamorphosis): Macro-based MeTTa to (Chicken) Scheme translator.
MIT License
7 stars 1 forks source link

Automated testing #21

Closed patham9 closed 11 months ago

patham9 commented 11 months ago
Necr0x0Der commented 11 months ago

It is more related to #17 , but I'd propose to have a clean folder for the library/extension itself, i.e. without files which are not needed to the library itself. Thus, I'd move example* and test.py outside extend, possibly, to tests for now

patham9 commented 11 months ago

I agree, hm how about ./extend/tests as it would separate it from the different kind of tests in the project root?

Necr0x0Der commented 11 months ago

I suppose that we will have automatic library/extension installation for MeTTa, which will suppose a certain structure for libraries. Like in Python, everything in the folder can possibly become a part of the library. In the nearest future, I suppose that the folder name with the library code should have the name of the library (e.g., mettamorph) with something like __init__.metta (or whatever else Luke will propose), but since it is not supported yet, I'm not proposing to rename extend. However, I'd put these tests into tests/extend rather than extend/tests.

patham9 commented 11 months ago

Sounds good, I will put them into tests/extend as you mentioned.

Necr0x0Der commented 11 months ago

Sounds good, I will put them into tests/extend as you mentioned.

Will you do it within this PR?

patham9 commented 11 months ago

I guess we can do it in a separate PR which could also address aspects of https://github.com/trueagi-io/metta-morph/issues/17 If that is fine then you can probably merge it already.