groovy / groovy/groovy-eclipse

Add compiler tests for built-in AST transforms

Open
#606 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue help wanted
Dominant language
Java
Stars
681
Forks
197
PR merge metrics
No merged PRs in 30d

Description

There are currently compiler tests for a few of the most venerable AST transforms. For the tested transforms, there could be many more cases covered (options that exist or have been added recently or other variations). For the untested transforms, tests should be added. This could also provide a good opportunity to learn how a particular transform works and can be applied.

Consult https://github.com/groovy/groovy-eclipse/blob/master/base-test/org.eclipse.jdt.groovy.core.tests.compiler/src/org/eclipse/jdt/groovy/core/tests/xform/package.html for an updated list of needs. Here is a snapshot at the time this issue was created:
xform-tests

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the updated needs list in base-test/org.eclipse.jdt.groovy.core.tests.compiler/src/org/eclipse/jdt/groovy/core/tests/xform/package.html, then inspect the existing compiler tests for AST transforms. Identify untested transforms and missing options or variations, and add coverage until the listed needs are addressed.

Written by the indexing model from the issue text.

Assessment

Tech stack
groovy, java
Domain
compilers, testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.