Fix Boolean nodes having lowercased token

Author: Juanjo Alvarez

driver/normalizer/tonode.go

@@ -83,7 +83,6 @@ var ToNode = &uast.ObjectToNode{
 		"Div":         "/",
 		"Ellipsis":    "...",
 		"Eq":          "==",
-		"False":       "false",
 		"For":         "for",
 		"FloorDiv":    "//",
 		"Global":      "global",
@@ -111,7 +110,6 @@ var ToNode = &uast.ObjectToNode{
 		"Return":      "return",
 		"RShift":      ">>",
 		"Sub":         "-",
-		"True":        "true",
 		"UAdd":        "+",
 		"USub":        "-",
 		"While":       "while",

fixtures/assert_constant.py.native

@@ -13,7 +13,7 @@
                     "lineno": 1,
                     "msg": null,
                     "test": {
-                        "LiteralValue": true,
+                        "LiteralValue": "True",
                         "ast_type": "BoolLiteral",
                         "col_offset": 8,
                         "end_col_offset": 11,
@@ -30,7 +30,7 @@
                     "lineno": 2,
                     "msg": null,
                     "test": {
-                        "LiteralValue": true,
+                        "LiteralValue": "True",
                         "ast_type": "BoolLiteral",
                         "col_offset": 8,
                         "end_col_offset": 11,
@@ -54,7 +54,7 @@
                         "s": "assert message"
                     },
                     "test": {
-                        "LiteralValue": true,
+                        "LiteralValue": "True",
                         "ast_type": "BoolLiteral",
                         "col_offset": 8,
                         "end_col_offset": 11,

fixtures/assert_constant.py.uast

@@ -2,7 +2,7 @@ Status:  ok
 Errors: 
 UAST: 
 Module {
-.  Roles: File,Module
+.  Roles: File
 .  Children: {
 .  .  0: Assert {
 .  .  .  Roles: Assert,Statement
@@ -23,7 +23,7 @@ Module {
 .  .  .  }
 .  .  .  Children: {
 .  .  .  .  0: BoolLiteral {
-.  .  .  .  .  Roles: Literal,Boolean,Expression,Primitive
+.  .  .  .  .  Roles: Literal,Boolean,Expression
 .  .  .  .  .  TOKEN "true"
 .  .  .  .  .  StartPosition: {
 .  .  .  .  .  .  Offset: 7
@@ -61,7 +61,7 @@ Module {
 .  .  .  }
 .  .  .  Children: {
 .  .  .  .  0: BoolLiteral {
-.  .  .  .  .  Roles: Literal,Boolean,Expression,Primitive
+.  .  .  .  .  Roles: Literal,Boolean,Expression
 .  .  .  .  .  TOKEN "true"
 .  .  .  .  .  StartPosition: {
 .  .  .  .  .  .  Offset: 20
@@ -98,7 +98,7 @@ Module {
 .  .  .  }
 .  .  .  Children: {
 .  .  .  .  0: Str {
-.  .  .  .  .  Roles: Literal,String,Expression,Primitive
+.  .  .  .  .  Roles: Literal,String,Expression
 .  .  .  .  .  TOKEN "assert message"
 .  .  .  .  .  StartPosition: {
 .  .  .  .  .  .  Offset: 38
@@ -115,7 +115,7 @@ Module {
 .  .  .  .  .  }
 .  .  .  .  }
 .  .  .  .  1: BoolLiteral {
-.  .  .  .  .  Roles: Literal,Boolean,Expression,Primitive
+.  .  .  .  .  Roles: Literal,Boolean,Expression
 .  .  .  .  .  TOKEN "true"
 .  .  .  .  .  StartPosition: {
 .  .  .  .  .  .  Offset: 32
@@ -137,3 +137,4 @@ Module {
 .  }
 }
 
+

fixtures/if.py.native

@@ -15,7 +15,7 @@
                             "value": {
                                 "args": [
                                     {
-                                        "LiteralValue": true,
+                                        "LiteralValue": "True",
                                         "ast_type": "BoolLiteral",
                                         "col_offset": 11,
                                         "end_col_offset": 14,
@@ -46,7 +46,7 @@
                     "lineno": 1,
                     "orelse": [],
                     "test": {
-                        "LiteralValue": true,
+                        "LiteralValue": "True",
                         "ast_type": "BoolLiteral",
                         "col_offset": 4,
                         "end_col_offset": 7,
@@ -65,7 +65,7 @@
                             "value": {
                                 "args": [
                                     {
-                                        "LiteralValue": true,
+                                        "LiteralValue": "True",
                                         "ast_type": "BoolLiteral",
                                         "col_offset": 11,
                                         "end_col_offset": 14,

native/python_package/python_driver/astimprove.py

@@ -486,7 +486,7 @@ def visit_NameConstant(self, node):
         if "value" in node:
             repr_val = repr(node["value"])
             if repr_val in ("True", "False"):
-                node.update({"LiteralValue": node["value"],
+                node.update({"LiteralValue": "True" if node["value"] else "False",
                              "ast_type": "BoolLiteral"})
             elif repr_val == "None":
                 node.update({"LiteralValue": "None",