Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

delete list comprehension #12392

Merged
merged 4 commits into from
Nov 22, 2019
Merged

delete list comprehension #12392

merged 4 commits into from
Nov 22, 2019

Conversation

krux02
Copy link
Contributor

@krux02 krux02 commented Oct 9, 2019

removal is overdue

@Araq
Copy link
Member

Araq commented Oct 10, 2019

No changelog entry?

@krux02
Copy link
Contributor Author

krux02 commented Oct 12, 2019

done

@krux02
Copy link
Contributor Author

krux02 commented Oct 14, 2019

The changelog entry cause the test failure? The failing test is surprisingly tsqlite.nim how? Is it a new flaky test? Should I disable it?

@krux02
Copy link
Contributor Author

krux02 commented Oct 16, 2019

Failure: reBuildFailed
package test failed
$ nim c -r tests/tsqlite.nim
Hint: used config file 'C:\projects\nim\config\nim.cfg' [Conf]
Hint: used config file 'C:\projects\nim\config\config.nims' [Conf]
Hint: used config file 'C:\projects\nim\pkgstemp\norm\tests\config.nims' [Conf]

[Suite] Creating and dropping tables, CRUD
    C:\projects\nim\pkgstemp\norm\tests\tsqlite.nim(82, 51): Check failed: dbConn.getAllRows(sql query % "users") ==
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
      @[?1, ?"email", ?"TEXT", ?1, ?nil, ?0],
      @[?2, ?"ssn", ?"INTEGER", ?0, ?nil, ?0],
      @[?3, ?"birthDate", ?"INTEGER", ?1, ?nil, ?0],
      @[?4, ?"lastLogin", ?"INTEGER", ?1, ?nil, ?0]]
    dbConn.getAllRows(sql query % "users") was @[@[0, 'id', 'INTEGER', 1, '0', 1], @[1, 'email', 'TEXT', 1, '''''', 0], @[2, 'ssn', 'INTEGER', 0, NULL, 0], @[3, 'birthDate', 'INTEGER', 1, NULL, 0], @[4, 'lastLogin', 'INTEGER', 1, '0', 0]]
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
  @[?1, ?"email", ?"TEXT", ?1, ?nil, ?0],
  @[?2, ?"ssn", ?"INTEGER", ?0, ?nil, ?0],
  @[?3, ?"birthDate", ?"INTEGER", ?1, ?nil, ?0],
  @[?4, ?"lastLogin", ?"INTEGER", ?1, ?nil, ?0]] was @[@[0, 'id', 'INTEGER', 1, NULL, 1], @[1, 'email', 'TEXT', 1, NULL, 0], @[2, 'ssn', 'INTEGER', 0, NULL, 0], @[3, 'birthDate', 'INTEGER', 1, NULL, 0], @[4, 'lastLogin', 'INTEGER', 1, NULL, 0]]
    C:\projects\nim\pkgstemp\norm\tests\tsqlite.nim(89, 51): Check failed: dbConn.getAllRows(sql query % "books") ==
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
      @[?1, ?"title", ?"TEXT", ?1, ?nil, ?0],
      @[?2, ?"authorEmail", ?"TEXT", ?1, ?nil, ?0],
      @[?3, ?"publisherTitle", ?"TEXT", ?1, ?nil, ?0]]
    dbConn.getAllRows(sql query % "books") was @[@[0, 'id', 'INTEGER', 1, '0', 1], @[1, 'title', 'TEXT', 1, '''''', 0], @[2, 'authorEmail', 'TEXT', 1, '''''', 0], @[3, 'publisherTitle', 'TEXT', 1, '''''', 0]]
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
  @[?1, ?"title", ?"TEXT", ?1, ?nil, ?0],
  @[?2, ?"authorEmail", ?"TEXT", ?1, ?nil, ?0],
  @[?3, ?"publisherTitle", ?"TEXT", ?1, ?nil, ?0]] was @[@[0, 'id', 'INTEGER', 1, NULL, 1], @[1, 'title', 'TEXT', 1, NULL, 0], @[2, 'authorEmail', 'TEXT', 1, NULL, 0], @[3, 'publisherTitle', 'TEXT', 1, NULL, 0]]
    C:\projects\nim\pkgstemp\norm\tests\tsqlite.nim(95, 54): Check failed: dbConn.getAllRows(sql query % "editions") ==
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
      @[?1, ?"title", ?"TEXT", ?1, ?nil, ?0],
      @[?2, ?"bookId", ?"INTEGER", ?1, ?nil, ?0]]
    dbConn.getAllRows(sql query % "editions") was @[@[0, 'id', 'INTEGER', 1, '0', 1], @[1, 'title', 'TEXT', 1, '''''', 0], @[2, 'bookId', 'INTEGER', 1, NULL, 0]]
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
  @[?1, ?"title", ?"TEXT", ?1, ?nil, ?0],
  @[?2, ?"bookId", ?"INTEGER", ?1, ?nil, ?0]] was @[@[0, 'id', 'INTEGER', 1, NULL, 1], @[1, 'title', 'TEXT', 1, NULL, 0], @[2, 'bookId', 'INTEGER', 1, NULL, 0]]
  [FAILED] Create tables
    C:\projects\nim\pkgstemp\norm\tests\tsqlite.nim(246, 51): Check failed: dbConn.getAllRows(sql query % "users") ==
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
      @[?1, ?"email", ?"TEXT", ?1, ?nil, ?0],
      @[?2, ?"ssn", ?"INTEGER", ?0, ?nil, ?0],
      @[?3, ?"birthDate", ?"INTEGER", ?1, ?nil, ?0],
      @[?4, ?"lastLogin", ?"INTEGER", ?1, ?nil, ?0]]
    dbConn.getAllRows(sql query % "users") was @[@[0, 'id', 'INTEGER', 1, '0', 1], @[1, 'email', 'TEXT', 1, '''''', 0], @[2, 'ssn', 'INTEGER', 0, NULL, 0], @[3, 'birthDate', 'INTEGER', 1, NULL, 0], @[4, 'lastLogin', 'INTEGER', 1, '0', 0]]
    @[@[?0, ?"id", ?"INTEGER", ?1, ?nil, ?1],
  @[?1, ?"email", ?"TEXT", ?1, ?nil, ?0],
  @[?2, ?"ssn", ?"INTEGER", ?0, ?nil, ?0],
  @[?3, ?"birthDate", ?"INTEGER", ?1, ?nil, ?0],
  @[?4, ?"lastLogin", ?"INTEGER", ?1, ?nil, ?0]] was @[@[0, 'id', 'INTEGER', 1, NULL, 1], @[1, 'email', 'TEXT', 1, NULL, 0], @[2, 'ssn', 'INTEGER', 0, NULL, 0], @[3, 'birthDate', 'INTEGER', 1, NULL, 0], @[4, 'lastLogin', 'INTEGER', 1, NULL, 0]]
    C:\projects\nim\pkgstemp\norm\tests\tsqlite.nim(253, 51): Check failed: dbConn.getAllRows(sql query % "books") ==
    @[@[?0, ?"id", ?"INTEGER

@krux02 krux02 closed this Oct 16, 2019
@krux02 krux02 reopened this Oct 16, 2019
@narimiran
Copy link
Member

The changelog entry cause the test failure? The failing test is surprisingly tsqlite.nim how? Is it a new flaky test? Should I disable it?

Can you resolve merge conflicts so we can then see if the failing test was some one time glitch, or if there is something else going on?

@narimiran narimiran merged commit 2acf74d into nim-lang:devel Nov 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants