Skip to content
This repository has been archived by the owner on Sep 7, 2023. It is now read-only.

Add some testing support for the REPL #279

Open
jeromesimeon opened this issue Jul 10, 2018 · 5 comments
Open

Add some testing support for the REPL #279

jeromesimeon opened this issue Jul 10, 2018 · 5 comments

Comments

@jeromesimeon
Copy link
Member

The REPL isn't being tested systematically.

@elissavetv
Copy link

Hiya! is this still open?

@ghost
Copy link

ghost commented Oct 9, 2019

i will work on this?can u explain this

@mttrbrts mttrbrts removed the Hacktoberfest by DigitalOcean and DEV label Nov 1, 2019
@beingOptimusP
Copy link

hey i would like to work on this issue, can u tell more about it?

@mttrbrts
Copy link
Member

hey i would like to work on this issue, can u tell more about it?

Of course! The Ergo REPL (also called ergotop) is a command line utility that allows a developer to use Ergo code interactive. See the REPL documentation for more details. https://docs.accordproject.org/docs/ergo-repl.html

Guidance for building the Ergo compiler and REPL locally are available here:
https://github.com/accordproject/ergo/blob/e297da1228b3342598300b5744fe1a34d52ae815/DEVELOPERS.md#build-the-ergo-compiler-and-repl

This issue requires adding some automated unit tests for the REPL to ensure that the REPL behaviour matches the behaviour of the engine on a particular platform. For example, I would recommend that you start by verifying that all of the examples in https://github.com/accordproject/ergo/blob/master/examples/repl_examples.ergo give the expected output in https://github.com/accordproject/ergo/blob/master/examples/repl_examples.result

One of the challenges with this issue is choosing a framework for running test suites against stdin and verifying the responses for stdout.

I you'd like to work on this, please confirm and I will assign to you.

@beingOptimusP
Copy link

beingOptimusP commented Mar 13, 2021

yeah sure, i am currently busy coz of endsems, will surely work on it after that, thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants