Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix map error reporting #365

Merged
merged 38 commits into from
Jun 25, 2020
Merged

Fix map error reporting #365

merged 38 commits into from
Jun 25, 2020

Conversation

afsalthaj
Copy link
Collaborator

@afsalthaj afsalthaj commented Jun 25, 2020

Make the error reporting path for map correct.

For example, given the below config:

case class Config(m2: Map[String, A])
case class A(a : Int, b: Int)

and a source HOCON as below:

  {
    m2 : {
        m21 : {
           b : 1
         }

        m22: {
           a : 2
        }

        m23: {
           a: 3
           b: 3
        }
    }
  }

then, the error reporting is, as given below:

ReadError:

╠══╦══╗
    
    ╠─MissingValue
     path: m2.m21.a
     Details: value of type int
    
  
  ╠══╗
    
    ╠─MissingValue
     path: m2.m22.b
     Details: value of type int
    
  

@afsalthaj afsalthaj merged commit a7266fe into master Jun 25, 2020
@afsalthaj afsalthaj deleted the fix_map_error branch July 8, 2020 17:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants