Hi Krishna —

Welcome to the project!

I think that the issues themselves are reasonable requests, though I also think there are some challenges to making the two issues you filed today be sufficient to support unit testing given other related challenges, like the ones that Michael raised in the comments of #8326.

I also wanted to mention that when we discussed various approaches to unit testing as a team a few weeks ago, there were definitely split opinions as to whether or not a class+method-based approach is preferable. Specifically, since Chapel isn't a pure OO language and significant codes can be written without using classes, there was a desire among some of the team members for a way to write unit tests that did not require creating a class. And if we had such a way, it raises the question for me as to whether that would be sufficient to test code that did use classes as well.

The combination of those two things would give me pause in diving into these implementation tasks without a clear picture of what the unit testing framework looks like, how it would work, and buy-in that it's what the team and users want. I know that Ben was wrestling with these questions before leaving the other week, but am not clear how far he got.

Best wishes,
-Brad


On Wed, 22 May 2019, Krishna Kumar Dey wrote:

Hey All,

I am Krishna Kumar Dey, GSoC'19 student developer where I am working on *Unit
Test Framework* for Chapel. While discussing with my mentors I came across
the reflection module that I will be using for the project. This module
lacks a few functionalities which we need for the development of the
framework. I have created issues for the same. And I need your opinion
regarding these feature requests.

Issues:

  - Proposed support for Reflection functions that get number of methods
  of a class. #13069 <https://github.com/chapel-lang/chapel/issues/13069>
  - Proposed support for Reflection functions that get name of method of a
  class. #13068 <https://github.com/chapel-lang/chapel/issues/13068>
  - Call methods from method name in string #8326
  <https://github.com/chapel-lang/chapel/issues/13069>


--
Thank you,
Krishna Kumar Dey

<https://github.com/chapel-lang/chapel/issues/13069>
_______________________________________________
Chapel-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/chapel-developers

Reply via email to