d5e7ee0d82
[tests] run tests using report.json and record all bugs + exceptions
...
Summary:
Instead of the custom filtering done by `InferPrint --issues-tests`, use the
filtering done by `infer` and run without filtering for our e2e tests. We still
test the filtering for our build systems integration tests, and this diff
restores that behaviour for the ant test (hence the bugs removed from
ant/issues.exp).
Also add internal exceptions to most tests to get more signal out of them (eg,
knowing when we add assertion failures and the like).
Retire the old `--issues-tests` to limit the number of ways we do filtering.
Reviewed By: jeremydubreil
Differential Revision: D4131308
fbshipit-source-id: 35805cc
8 years ago
c9bac51b81
[tests] make clang tests Makefile more functional
...
Summary:
This makes the tests depend on much fewer phony targets, thus reducing the need
to rerun the tests when nothing has changed.
Reviewed By: jberdine
Differential Revision: D4118457
fbshipit-source-id: 664b6e3
8 years ago
a71902355f
[debug][dotty] Fix issue in dotty output where overloaded functions were conflated
...
Reviewed By: jvillard
Differential Revision: D4117078
fbshipit-source-id: fdd8e93
8 years ago
aac14999b0
[componentkit] Whitelist Native Templates
...
Reviewed By: tks2103
Differential Revision: D4090237
fbshipit-source-id: 63f233e
8 years ago
a45d59e478
[componentkit] Don't count unavailable initializers
...
Reviewed By: tks2103
Differential Revision: D4090154
fbshipit-source-id: e0626e6
8 years ago
3fb8801b6c
[IR] Change cfg representation so the node number is per-procedure and not per-cfg
...
Reviewed By: jeremydubreil
Differential Revision: D4088075
fbshipit-source-id: 6e517a7
8 years ago
5752b14d9c
[componentkit] Skip implicit variables during MLV check
...
Reviewed By: dulmarod
Differential Revision: D4084601
fbshipit-source-id: 1c919cd
8 years ago
5580be8dcc
[componentkit] Initializer With Side-Effect Analyzer
...
Reviewed By: dulmarod
Differential Revision: D4063217
fbshipit-source-id: 7196a64
8 years ago
f45cf115e6
[test] Do not use infer from PATH
...
Summary:
In several places the tests were using whatever 'infer' executable was
found in PATH, instead of the one build from the source to be tested.
Reviewed By: jeremydubreil
Differential Revision: D4065019
fbshipit-source-id: 9b65099
8 years ago
c2a8dae26f
[componentkit] Multiple factory methods
...
Reviewed By: dulmarod
Differential Revision: D4033747
fbshipit-source-id: cf2801b
8 years ago
e73350598a
[componentkit] Unconventional superclass rule
...
Reviewed By: dulmarod
Differential Revision: D3954639
fbshipit-source-id: 5d61c66
8 years ago
a4501462cc
[componentkit] Reference fake header in build system test
...
Reviewed By: dulmarod
Differential Revision: D4033937
fbshipit-source-id: adccbd6
8 years ago
62bfde8b5a
[clang] translate global var initializers
...
Summary:
Create dummy functions representing the initializers of global variables. This
is so we can implement checks in the backend that can look at the initializer
expressions of global variables. We try not to create these dummy functions
when the initializer is not present, although for some reason we sometimes end
up with empty initializers.
Also add source file info to global variables in the backend (Pvar.re).
Reviewed By: sblackshear
Differential Revision: D3780238
fbshipit-source-id: 2dca87e
8 years ago
6697ed781f
[tests] Rerun test without swallowing output on failure
...
Reviewed By: jvillard
Differential Revision: D3992995
fbshipit-source-id: 85d2586
8 years ago
7d49e16d63
[clang] rewrite hijack_and_normalize_clang_command in OCaml
...
Summary:
Let's start migrating some of our bash script to OCaml to make them easier to
maintain and extend.
For now replace just one script and put it in lib/clang_wrappers/ at compile
time, where the former script used to be. Further simplifications will come
later.
Reviewed By: jberdine
Differential Revision: D3929988
fbshipit-source-id: b2d8b37
8 years ago
272c21ff70
[componentkit] Factory functions analyzer
...
Reviewed By: dulmarod
Differential Revision: D3933693
fbshipit-source-id: 6294fb4
8 years ago
967532ea2e
[componentkit][test] Rename test files
...
Reviewed By: dulmarod
Differential Revision: D3924307
fbshipit-source-id: 77ddfac
8 years ago
df988dd051
[componentkit][test] Import proper fake header in second test
...
Reviewed By: dulmarod
Differential Revision: D3924245
fbshipit-source-id: 16a110a
8 years ago
6475e358c4
[componentkit][const] Ignore NSError
...
Reviewed By: martinoluca
Differential Revision: D3912232
fbshipit-source-id: adc8a55
8 years ago
7a434adea8
[componentkit][const] Whitelist certain classes
...
Reviewed By: martinoluca
Differential Revision: D3911399
fbshipit-source-id: c523779
8 years ago
d2ac2242c4
[componentkit][test] typedef struct -> class
...
Reviewed By: dulmarod
Differential Revision: D3909628
fbshipit-source-id: 52e9539
8 years ago
f54b84c751
[componentkit][test] Move mock CK imports to own .h file
...
Reviewed By: dulmarod
Differential Revision: D3909240
fbshipit-source-id: 66e8b71
8 years ago
561f4fdfad
[linters] Adding a new type of procname that is used for lint bugs that are not in any method
...
Reviewed By: cristianoc
Differential Revision: D3913535
fbshipit-source-id: cb8f840
8 years ago
847c141912
[tests] Clean up test files shared between frontend and endtoend tests
...
Reviewed By: jberdine
Differential Revision: D3893900
fbshipit-source-id: 497effc
8 years ago
c134eac91e
[tests] Convert objective-c frontend tests to new direct format
...
Reviewed By: jberdine
Differential Revision: D3886011
fbshipit-source-id: 2e0c014
8 years ago
fe21442d47
[tests] Convert the linter tests to the new direct format
...
Reviewed By: jberdine
Differential Revision: D3877153
fbshipit-source-id: a0efb3c
8 years ago
e19550a43b
Handle l-value refs correctly
...
Reviewed By: jvillard
Differential Revision: D3757766
fbshipit-source-id: 12edcfc
8 years ago
9ae19d01d0
Make the is_global_var not fire for the static local variables
...
Reviewed By: jberdine, ryanrhee
Differential Revision: D3741764
fbshipit-source-id: 42be7f7
8 years ago
40b176fd01
Translate global const vars locally to a procedure
...
Summary: With this approach, all the global consts will be inlined in the places where they are used.
Reviewed By: dulmarod
Differential Revision: D3703133
fbshipit-source-id: 3c19479
8 years ago
e81e43456b
Test mutable local variables in CK files
...
Reviewed By: jvillard
Differential Revision: D3694032
fbshipit-source-id: c06b049
8 years ago
d594a525ac
Test that CK rules don't fire in non-component files
...
Reviewed By: jvillard
Differential Revision: D3694101
fbshipit-source-id: 1c2913f
8 years ago
e2ff6e58a5
`buck test componentkit` boilerplate
...
Reviewed By: jvillard
Differential Revision: D3693997
fbshipit-source-id: de64bbe
8 years ago
8b0b485921
Refactoring of tests, separating the tests for linters
...
Reviewed By: sblackshear
Differential Revision: D3569981
fbshipit-source-id: 198647b
9 years ago
fd8a864c15
doing preanalysis on-demand
...
Reviewed By: jeremydubreil
Differential Revision: D3352767
fbshipit-source-id: a9dcc0a
9 years ago
617ffab0ac
Add @generated comment to icfg.dot files
...
Reviewed By: jvillard
Differential Revision: D3358243
fbshipit-source-id: a47cc01
9 years ago
aae1dbac3d
Skipping reporting global variable warning on known non-expensive cases.
...
Reviewed By: dulmarod
Differential Revision: D3298191
fbshipit-source-id: 8b3a38b
9 years ago
20925df57c
removing unused deallocate param in nullify instr
...
Reviewed By: jeremydubreil
Differential Revision: D3263241
fbshipit-source-id: b0d2c0f
9 years ago
3defb52e4e
Fixing broken test + adding objcpp tests to make test target
...
Reviewed By: jvillard
Differential Revision: D3270395
fb-gh-sync-id: 8972e8e
fbshipit-source-id: 8972e8e
9 years ago
83c1bbc832
Implementing a checker to warn initialization of global variables with mehod calls.
...
Reviewed By: dulmarod
Differential Revision: D3184584
fb-gh-sync-id: dbfc99c
fbshipit-source-id: dbfc99c
9 years ago
2fb2b3d9e1
Add the virtual flag of C++ methods in method calls
...
Reviewed By: akotulski
Differential Revision: D2943884
fb-gh-sync-id: 33b1c8b
shipit-source-id: 33b1c8b
9 years ago
4584f7f6fc
[clang-format] Reformat all c/cpp/objc sources with clang-format
...
Reviewed By: jul
Differential Revision: https://phabricator.fb.com/D2953843
9 years ago
d4d5c3bb2f
Implemented new check CXX_REFERENCE_CAPTURED_IN_OBJC_BLOCK
...
Reviewed By: dulmarod
Differential Revision: D2823705
fb-gh-sync-id: 594bfa2
9 years ago
63b57790f9
Fix indentation of multiline copyright comment
...
Summary:
public
Lines other than the first of multi-line comments in non-ocaml files
were flush right instead of aligned.
Reviewed By: jvillard
Differential Revision: D2739752
fb-gh-sync-id: c85f56e
9 years ago
ca613a77ff
[copyrightor] format start and end of copyright comments
...
Summary:
The script now reformats the start and end of comments if necessary.
9 years ago
3e199467f4
Complete copyright messages.
10 years ago
b8982270f2
initial synchronization
10 years ago
Jules Villard
Cristiano Calcagno
Ryan Rhee
Josh Berdine
Dulma Churchill
Martino Luca
Sam Blackshear
Andrzej Kotulski
Dino Distefano
Open Source Synchronization